Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21678 )
Change subject: IMPALA-13298: Correct test_miss_complete_cb Assertion ...................................................................... IMPALA-13298: Correct test_miss_complete_cb Assertion Commit ad868b9 modified the RPC error messages to include the query id. Part of this change was to remove the string "Query aborted" from the test's beeswax client when it raises an error. The TestRPCTimeout.test_miss_complete_cb was missed and was still asserting on the string "Query aborted". This patch fixes that test to use the new error_msg_expected() function. Testing was accomplished by running the TestRPCTimeout.test_miss_complete_cb test locally. Change-Id: Ifb2fa3864b52b0927c6488b22135913b90bce3bd Reviewed-on: http://gerrit.cloudera.org:8080/21678 Reviewed-by: Riza Suminto <[email protected]> Reviewed-by: Xuebin Su <[email protected]> Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M tests/custom_cluster/test_rpc_timeout.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Riza Suminto: Looks good to me, but someone else must approve Xuebin Su: Looks good to me, but someone else must approve Michael Smith: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21678 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifb2fa3864b52b0927c6488b22135913b90bce3bd Gerrit-Change-Number: 21678 Gerrit-PatchSet: 3 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Xuebin Su <[email protected]>
