Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16157 )
Change subject: IMPALA-5534: Fix and enable experimental failure tests ...................................................................... IMPALA-5534: Fix and enable experimental failure tests Moves the test_catalog_hms_failures.py and test_process_failures.py from the experimental tests to custom cluster tests. catalog_service/test_hms_failure.py is combined with custom_cluster/test_catalog_hms_failure.py as well in order to unify all tests for HMS failures. Several modifications to the tests were necessary to get them working again, but for the most part, the logic of the tests remained the same. A few additional fault tolerance tests (e.g. TestHiveMetaStoreFailure::test_hms_client_retries) were added as well. The overall goal is to increase the process failure test coverage for all components: impalads, statestore, catalogd, HMS, etc. test_restart_catalogd in test_process_failures.py fails due to IMPALA-9848, so it is skipped for now. Testing: * Ran new tests locally Change-Id: I9dbb98017fb6c40cea349e7c63a35c325cbbc288 Reviewed-on: http://gerrit.cloudera.org:8080/16157 Reviewed-by: Sahil Takiar <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/service/Frontend.java D tests/catalog_service/test_hms_failure.py M tests/common/impala_service.py A tests/custom_cluster/test_catalog_hms_failures.py A tests/custom_cluster/test_process_failures.py D tests/experiments/test_catalog_hms_failures.py D tests/experiments/test_process_failures.py 7 files changed, 485 insertions(+), 466 deletions(-) Approvals: Sahil Takiar: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16157 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9dbb98017fb6c40cea349e7c63a35c325cbbc288 Gerrit-Change-Number: 16157 Gerrit-PatchSet: 5 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
