Sahil Takiar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16149


Change subject: IMPALA-9834: De-flake TestQueryRetries on EC builds
......................................................................

IMPALA-9834: De-flake TestQueryRetries on EC builds

This patch makes the following changes to de-flake TestQueryRetries on
EC builds.

It skips any test that scans tpch.lineitem (referred to as a
the _shuffle_heavy_query in TestQueryRetries). This query runs on three
instances during regular builds (HDFS, S3, etc.), but only two instances
on EC builds. This causes some non-deterministism during the test becase
killing an impalad in the mini-cluster won't necessarily cause a retry
to be triggered.

It bumps up the timeout used when waiting for a query to be retried.

It improves the assertion in __get_query_id_from_profile so that it
dumps the full profile when the assertion fails. This should help
debuggability of any test failures that fail in this assertion.

Testing:
* Ran TestQueryRetries locally

Change-Id: Id5c73c2cbd0ef369175856c41f36d4b0de4b8d71
---
M tests/custom_cluster/test_query_retries.py
1 file changed, 11 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/16149/1
--
To view, visit http://gerrit.cloudera.org:8080/16149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5c73c2cbd0ef369175856c41f36d4b0de4b8d71
Gerrit-Change-Number: 16149
Gerrit-PatchSet: 1
Gerrit-Owner: Sahil Takiar <stak...@cloudera.com>

Reply via email to