Hello Quanlong Huang, Riza Suminto, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-13690: Fix flaky test 
test_get_operation_status_for_async_ddl
......................................................................

IMPALA-13690: Fix flaky test test_get_operation_status_for_async_ddl

This test only worked as intended with BEESWAX protocol, because in
case of HS2 executing "set opt=?" does not affect execute_async().
As the sleep in the debug_action had no effect, the query could
reach its terminal state too quickly. Besides fixing query option
handling also added a sleep to the query itself to keep it longer
in RUNNING state.

Also fixed test_concurrent_alter_table_rename which didn't apply
sync_ddl=1 for the same reason.

Testing:
- looped test_get_operation_status_for_client 100 times with debug
  and ASAN builds

Change-Id: I260e40a7cdeae57f18891e009f32a9466b2f8ece
---
M tests/metadata/test_ddl.py
1 file changed, 14 insertions(+), 31 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/08/23308/4
--
To view, visit http://gerrit.cloudera.org:8080/23308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I260e40a7cdeae57f18891e009f32a9466b2f8ece
Gerrit-Change-Number: 23308
Gerrit-PatchSet: 4
Gerrit-Owner: Csaba Ringhofer <[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