Hello Quanlong Huang, Csaba Ringhofer, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/23360

to look at the new patch set (#2).

Change subject: IMPALA-13731: Deflake TestAsyncLoadData.test_async_load
......................................................................

IMPALA-13731: Deflake TestAsyncLoadData.test_async_load

This patch attempt to deflake TestAsyncLoadData::test_async_load by
allowing exec_end_state == PENDING if enable_async_load_data. This is OK
because, if enable_async_load_data is true, async_exec_thread_ might be
slightly slower to start and transition the query to RUNNING. wait_time
is also relaxed to at least 2 seconds because wait_start not strictly
starts at the point when the async_exec_thread_ starts.

This patch does not change assertion / behavior in sync code
path (enable_async_ddl_execution=false).

Testing:
Loop and pass TestAsyncLoadData 50 times.

Change-Id: I8776a432e60a1167ad54778e81046421df15cf37
---
M tests/metadata/test_load.py
1 file changed, 6 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/60/23360/2
--
To view, visit http://gerrit.cloudera.org:8080/23360
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8776a432e60a1167ad54778e81046421df15cf37
Gerrit-Change-Number: 23360
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to