Re: Idea: Remove PowerMock

+1 Kill it! :) mockito is a more modern approach. I think it is cool that we modernize our platform, and remove old and unsupported tools and components. On Mon, Jul 10, 2023 at 5:36 PM Ayush Saxena wrote: > +1, PowerMock as far as I remember has issues with JDK-11+ as well, > one such ref : >

Re: Is there a way to test schema evolution scripts?

Hello, The tests under DbInstallBase [1] seem to do more or less what you want. Check them out and if something is missing don't hesitate to enrich them. Best, Stamatis [1] https://github.com/apache/hive/blob/5e46e80bc7d059093aece81e3886ba5ee425ee95/standalone-metastore/metastore-server/src/tes

Re: Is there a way to test schema evolution scripts?

Hi Zsolt, You can start by exploring this: https://github.com/apache/hive/blob/master/standalone-metastore/DEV-README It has steps to run those metatore scripts over different database types locally. -Ayush On Mon, 10 Jul 2023 at 20:30, Zsolt Miskolczi wrote: > > Hi, > > I just stared at a pull

Re: Idea: Remove PowerMock

+1, PowerMock as far as I remember has issues with JDK-11+ as well, one such ref : https://stackoverflow.com/questions/52966897/powermock-java-11 -Ayush On Mon, 10 Jul 2023 at 20:18, Zsolt Miskolczi wrote: > > Hi, > > Hive heavily uses PowerMock . The main > purpose

Is there a way to test schema evolution scripts?

Hi, I just stared at a pull request about using liquibase for schema evolution and I was thinking how such a change can be validated. I think the main issue is that we support multiple types of relational databases but precommit tests don't test them at all. I think if we want to check if the id

Idea: Remove PowerMock

Hi, Hive heavily uses PowerMock . The main purpose of it is having static mocking. The sad thing is it seems PowerMock is dead: - The main branch got it's lot commit in 2022 and and most of the contributions last year were simple dependency upgrades: https://github.c