Hello Zoltan Borok-Nagy,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21742
to look at the new patch set (#2).
Change subject: IMPALA-11431: Avoid getting stats for synthetic column row__id
from HMS
......................................................................
IMPALA-11431: Avoid getting stats for synthetic column row__id from HMS
Before this change Impala always tried to fetch stats for row__id in
full ACID tables, which does not exist in the metastore. Sometimes this
led to an exception from HMS, see HIVE-28498 for details.
This caused flakyness in test_compute_stats_with_structs. The test also
had side effects (it computed stats for a shared table) so it was
modified to use unique_database.
Testing:
- could reproduce the issue by starting HMS with
hive.metastore.try.direct.sql=true and verified that the change
fixes it
Change-Id: I759f57c99aa16e4ab5fd82aa5f6b756446291f03
---
M fe/src/main/java/org/apache/impala/catalog/CtasTargetTable.java
M fe/src/main/java/org/apache/impala/catalog/FeTable.java
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M fe/src/main/java/org/apache/impala/catalog/KuduTable.java
M fe/src/main/java/org/apache/impala/catalog/Table.java
M fe/src/main/java/org/apache/impala/catalog/VirtualTable.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalTable.java
M
testdata/workloads/functional-query/queries/QueryTest/compute-stats-with-structs.test
M tests/query_test/test_nested_types.py
9 files changed, 50 insertions(+), 28 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21742/2
--
To view, visit http://gerrit.cloudera.org:8080/21742
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I759f57c99aa16e4ab5fd82aa5f6b756446291f03
Gerrit-Change-Number: 21742
Gerrit-PatchSet: 2
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
