+1 to running automated tests whenever a patch introduces schema changes. This
could act as safeguard in case a contributor forget to run these tests.
I think the extra running time is worth the testing in this case.
Vineet
> On Feb 13, 2018, at 3:48 PM, Vihang Karajgaonkar wrote:
>
> We used
We used to have a job which ran some tests when there were schema changes
in the patch. It ran scripts from testutils/metastore to setup and run some
basic validation checks (primarily to detect incompatible schema changes
and schema upgrades). I think we should run similar tests whenever schema
ch
They aren’t currently. I don’t know if we want to add them as part of it
or not. They take a little while to run in a fresh environment like the
precommit testing because they have to download the docker images. The
Oracle one is 2.4G. We can set it up to do it if we want to. Or maybe we
could
Hi Alan,
Will these tests be run as part of pre commit testing job?
Thanks,
Vineet G
> On Feb 12, 2018, at 9:56 AM, Alan Gates wrote:
>
> With the commit of HIVE-17983 to master there are now tests for install and
> upgrade scripts for the metastore database for all 4 of the database types
> (