Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23347 )
Change subject: IMPALA-14306: Frontend.getDataSrcs() should retry for InconsistentMetadataFetchException ...................................................................... IMPALA-14306: Frontend.getDataSrcs() should retry for InconsistentMetadataFetchException After IMPALA-14283, coordinator will throw an InconsistentMetadataFetchException when witness catalogd service ID changes. The Frontend code should retry the request to use fresh metadata. This patch resolves the TODO item in Frontend.getDataSrcs() to add the retry as other methods do. Also change the catch clause in LocalCatalog.getDataSources() so InconsistentMetadataFetchException can be thrown from it. Tests: - Ran TestExtDataSources.test_catalogd_ha_failover 100 times. Change-Id: I483423680a5c953aaf3446b62c8b8ee08d6c6385 Reviewed-on: http://gerrit.cloudera.org:8080/23347 Reviewed-by: Wenzhe Zhou <[email protected]> Reviewed-by: Riza Suminto <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/local/LocalCatalog.java M fe/src/main/java/org/apache/impala/service/Frontend.java 2 files changed, 12 insertions(+), 6 deletions(-) Approvals: Wenzhe Zhou: Looks good to me, but someone else must approve Riza Suminto: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/23347 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I483423680a5c953aaf3446b62c8b8ee08d6c6385 Gerrit-Change-Number: 23347 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
