Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15026
to look at the new patch set (#2).
Change subject: IMPALA-9068: Use different directories for external vs managed
warehouse
......................................................................
IMPALA-9068: Use different directories for external vs managed warehouse
Hive 3 changed the typical storage model for tables to split them
between two directories:
- hive.metastore.warehouse.dir stores managed tables (which is now
defined to be only transactional tables)
- hive.metastore.warehouse.external.dir stores external tables
(everything that is not a transactional table)
In more recent commits of Hive, there is now validation that the
external tables cannot be stored in the managed directory. In order
to adopt these newer versions of Hive, we need to use separate
directories for external vs managed warehouses.
Most of our test tables are not transactional, so they would reside
in the external directory. To keep the test changes small, this uses
/test-warehouse for the external directory and /test-warehouse/managed
for the managed directory. Having the managed directory be a subdirectory
of /test-warehouse means that the data snapshot code should not need to
change.
The Hive 2 configuration doesn't change as it does not have this concept.
Since this changes the dataload layout, this also sets the CDH_MAJOR_VERSION
to 7 for USE_CDP_HIVE=true. This means that dataload will uses a separate
location for data as compared to USE_CDP_HIVE=false. That should reduce
conflicts between the two configurations.
Testing:
- Ran exhaustive tests with USE_CDP_HIVE=false
- Ran exhaustive tests with USE_CDP_HIVE=true (with current Hive version)
- Verified that dataload succeeds and tests are able to run with a newer
Hive version.
Change-Id: I3db69f1b8ca07ae98670429954f5f7a1a359eaec
---
M bin/impala-config.sh
M fe/src/test/resources/hive-site.xml.py
M testdata/avro_schema_resolution/create_table.sql
M testdata/bin/load_nested.py
A testdata/cluster/node_templates/cdh7/etc/init.d/kms
M testdata/workloads/functional-query/queries/QueryTest/acid-compaction.test
M testdata/workloads/functional-query/queries/QueryTest/create-database.test
M testdata/workloads/functional-query/queries/QueryTest/describe-db.test
M testdata/workloads/functional-query/queries/QueryTest/describe-hive-db.test
M tests/common/environ.py
M tests/common/impala_test_suite.py
M tests/comparison/cluster.py
M tests/custom_cluster/test_hive_parquet_codec_interop.py
M tests/query_test/test_compressed_formats.py
14 files changed, 133 insertions(+), 38 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/15026/2
--
To view, visit http://gerrit.cloudera.org:8080/15026
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3db69f1b8ca07ae98670429954f5f7a1a359eaec
Gerrit-Change-Number: 15026
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
