Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/24320 )

Change subject: IMPALA-15003: More robust way to verify operator details
......................................................................

IMPALA-15003: More robust way to verify operator details

test_cancelled_nodes_in_exec_summary verifies that operator details have
expected CANCELLED markers. However, the row indexes of the operators
are hard-coded, making the test fragile across builds in different
environments.

This improves the test to iterate through the exec summary with a map of
operator names to their expected details. If a row matches the operator
name, verifies its details.

Testing
 - Ran the test locally and on Ozone erasure-coding build where the test
   failed.

Change-Id: I50f1236a2bb7a360e474db8fa42089303f734e79
Reviewed-on: http://gerrit.cloudera.org:8080/24320
Reviewed-by: Michael Smith <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M tests/common/impala_test_suite.py
M tests/query_test/test_observability.py
2 files changed, 37 insertions(+), 75 deletions(-)

Approvals:
  Michael Smith: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/24320
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I50f1236a2bb7a360e474db8fa42089303f734e79
Gerrit-Change-Number: 24320
Gerrit-PatchSet: 3
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>

Reply via email to