Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24135 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/24135 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Michael Smith <[email protected]> --- M tests/authorization/test_ranger.py 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Verified Michael Smith: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I538c30bf4d1439108bd1cd1cb64208281974c1f6 Gerrit-Change-Number: 24135 Gerrit-PatchSet: 5 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]>
