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


Change subject: IMPALA-8907: TestResultSpooling.test_slow_query is flaky
......................................................................

IMPALA-8907: TestResultSpooling.test_slow_query is flaky

De-flake test_result_spooling.py::TestResultsSpooling::test_slow_query
by increasing the delay in RowBatch production. This patch makes two
fixes to ensure that RowBatchGetWaitTime is a non-zero value:

* Add the DELAY DEBUG_ACTION to the SCAN_NODE rather than the
EXCHANGE_NODE. Since the EXCHANGE_NODE only processes a few rows, adding
the delay to the SCAN_NODE decreases the rate at which results are
produced.

* Wait for all rows to be fetched before checking if RowBatchGetWaitTime
is in the profile. This fixes a possible race condition where the fetch
thread was not able to issue the fetch RPC before the test checked if
RowBatchGetWaitTime was in the runtime profile.

Testing:
* Looped test_slow_query for several hours with 0 failures

Change-Id: Idcb4a6b38f85a9497f80e2674e1c1fa512be5940
---
M tests/query_test/test_result_spooling.py
1 file changed, 4 insertions(+), 3 deletions(-)



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

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

Reply via email to