Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24600 )
Change subject: IMPALA-15121: Fix query failure when catalogd is in standby mode during HA failover ...................................................................... IMPALA-15121: Fix query failure when catalogd is in standby mode during HA failover During a catalogd HA failover, an impalad coordinator might try to fetch metadata from a catalogd that was previously in active mode but has just restarted and in standby mode now. In this case, the metadata request is rejected with a standby mode error, causing the query to fail. This change recognizes the standby mode rejection error and converts it into an InconsistentMetadataFetchException, allowing the existing Frontend retry mechanism to retry the metadata fetch. As a result, the query retries against the new active catalogd at the end instead of failing. Added thrift error code CATALOG_IN_STANDBY_MODE for rejection on standby mode. Tests: Added a new ee test test_standby_retry_during_failover. Passed core tests. Change-Id: Ia511e22a596733e6a2dd752894d6da43e91faca6 Reviewed-on: http://gerrit.cloudera.org:8080/24600 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/catalog/catalog-server.cc M common/thrift/generate_error_codes.py M fe/src/main/java/org/apache/impala/catalog/local/CatalogdMetaProvider.java M tests/custom_cluster/test_catalogd_ha.py 4 files changed, 50 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24600 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia511e22a596733e6a2dd752894d6da43e91faca6 Gerrit-Change-Number: 24600 Gerrit-PatchSet: 5 Gerrit-Owner: Yida Wu <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]>
