Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22468 )
Change subject: IMPALA-13729: Accept error messages not starting with prompt ...................................................................... IMPALA-13729: Accept error messages not starting with prompt Previously, error_msg_expected() only accepted error messages starting with the following error prompt: ``` Query <query_id> failed:\n ``` However, for some tests using the Beeswax protocol, the error prompt may appear in the middle of the error message instead of at its beginning. Therefore, this patch adapts error_msg_expected() to accept error messages not starting with the error prompt. The error_msg_expected() function is renamed to error_msg_startswith() to better describe its behavior. Change-Id: Iac3e68bcc36776f7fd6cc9c838dd8da9c3ecf58b Reviewed-on: http://gerrit.cloudera.org:8080/22468 Reviewed-by: Daniel Becker <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Riza Suminto <[email protected]> --- M tests/common/impala_test_suite.py M tests/common/test_result_verifier.py M tests/custom_cluster/test_coordinators.py M tests/custom_cluster/test_executor_groups.py M tests/custom_cluster/test_kill_query.py M tests/custom_cluster/test_kudu.py M tests/custom_cluster/test_process_failures.py M tests/custom_cluster/test_rpc_timeout.py M tests/custom_cluster/test_sys_db.py M tests/hs2/hs2_test_suite.py M tests/query_test/test_iceberg.py M tests/shell/test_shell_commandline.py M tests/shell/test_shell_interactive.py M tests/util/cancel_util.py 14 files changed, 67 insertions(+), 60 deletions(-) Approvals: Daniel Becker: Looks good to me, but someone else must approve Impala Public Jenkins: Verified Riza Suminto: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22468 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iac3e68bcc36776f7fd6cc9c838dd8da9c3ecf58b Gerrit-Change-Number: 22468 Gerrit-PatchSet: 13 Gerrit-Owner: Xuebin Su <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Xuebin Su <[email protected]>
