Hello Mihaly Szjatinya, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24135
to look at the new patch set (#4).
Change subject: IMPALA-12715: deflake test_allow_metadata_update_local_catalog
......................................................................
IMPALA-12715: deflake test_allow_metadata_update_local_catalog
The test expects DESCRIBE on an unloaded table can load the table meta
in coordinator side. However, this can't be guaranteed in the local
catalog mode since catalog updates(invalidations) from statestore could
invalidate the cache (IMPALA-14856).
The test is flaky since DESCRIBE uses the table meta twice, one in
analyze phase (StmtMetadataLoader) and the other in execution
(Frontend.doDescribeTable()). If the statestore update arrives between
them, the second usage can load the table meta again so the cache item
exists. However, if the statestore update arrives after the second
usage, the cache item is removed so the test fails.
This patch deflakes the test by using REFRESH on the unloaded table to
trigger metadata loading in catalogd side. Run it with sync_ddl=true to
make sure coordinator processes the statestore update. Then run a
DESCRIBE to make sure the table meta is loaded in coordinator side.
Tests:
- ran the test locally 200 times.
- ran test_ranger.py
Change-Id: I538c30bf4d1439108bd1cd1cb64208281974c1f6
---
M tests/authorization/test_ranger.py
1 file changed, 9 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/24135/4
--
To view, visit http://gerrit.cloudera.org:8080/24135
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I538c30bf4d1439108bd1cd1cb64208281974c1f6
Gerrit-Change-Number: 24135
Gerrit-PatchSet: 4
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
