Quanlong Huang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22817 )
Change subject: IMPALA-13993: waitForHmsEvent should check table events under missing dbs ...................................................................... IMPALA-13993: waitForHmsEvent should check table events under missing dbs When a db is missing in the catalog cache, waitForHmsEvent request currently just check if there are pending database events on it, assuming processing the CREATE_DATABASE event will add the db with the table list. However, that's wrong since processing CREATE_DATABASE event just adds the db with an empty table list. We should still wait for pending events on underlying tables. Tests: - Added e2e test which consistenly fails when running concurrently with other tests in TestEventSyncWaiting without the fix. Change-Id: I3fe74fcf0bf4dbac4a3584f6603279c0a2730b0c Reviewed-on: http://gerrit.cloudera.org:8080/22817 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Quanlong Huang <huangquanl...@gmail.com> --- M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java M tests/metadata/test_event_processing.py 2 files changed, 16 insertions(+), 7 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Quanlong Huang: Verified -- To view, visit http://gerrit.cloudera.org:8080/22817 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3fe74fcf0bf4dbac4a3584f6603279c0a2730b0c Gerrit-Change-Number: 22817 Gerrit-PatchSet: 4 Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
