You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2019/04/30 20:45:25 UTC

[Impala-ASF-CR] WIP ONLY - IMPALA-8121: part 1: use local catalog in containers

Hello Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/13122

to look at the new patch set (#6).

Change subject: WIP ONLY - IMPALA-8121: part 1: use local catalog in containers
......................................................................

WIP ONLY - IMPALA-8121: part 1: use local catalog in containers

This enables "modern" catalog features including the
local catalog and HMS notification support in the
dockerised minicluster by default.

The flags can be overridden if needed.

Test fixes:
* Detect whether catalog_v2 is enabled from web UI.
* test_describe_db waits for metadata event processor to pick up new
  database and doesn't need to change database owner
* TestWebPage.test_catalog handles an expected exception from
  the /catalog_objects page on the impalad.
* test_pull_stats_profile: feature disabled with local catalog
* test_hms_service_dies: invalidate the test table instead of
  the whole catalog.
* test_compute_stats: Avro schema resolution behaviour changed
  with local catalog - IMPALA-7308

TODO:
* IMPALA-8458
* IMPALA-8459
* IMPALA-7131 (data sources)
* getTables() doesn't return comment

Change-Id: I060f2076da74fbbe92ae26dbad51f09a3bd20169
---
M docker/catalogd/Dockerfile
M docker/coord_exec/Dockerfile
M docker/coordinator/Dockerfile
M fe/src/main/java/org/apache/impala/catalog/local/LocalFsTable.java
A testdata/workloads/functional-query/queries/QueryTest/compute-stats-avro-catalog-v2.test
A testdata/workloads/functional-query/queries/QueryTest/compute-stats-avro.test
M testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
M testdata/workloads/functional-query/queries/QueryTest/describe-db.test
M tests/catalog_service/test_hms_failure.py
M tests/common/environ.py
M tests/common/impala_test_suite.py
M tests/common/skip.py
M tests/metadata/test_compute_stats.py
M tests/metadata/test_hms_integration.py
M tests/metadata/test_metadata_query_statements.py
M tests/metadata/test_refresh_partition.py
M tests/query_test/test_avro_schema_resolution.py
M tests/webserver/test_web_pages.py
18 files changed, 882 insertions(+), 402 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/22/13122/6
-- 
To view, visit http://gerrit.cloudera.org:8080/13122
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I060f2076da74fbbe92ae26dbad51f09a3bd20169
Gerrit-Change-Number: 13122
Gerrit-PatchSet: 6
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>