Csaba Ringhofer has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23308 )

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
Reviewed-on: http://gerrit.cloudera.org:8080/23308
Reviewed-by: Csaba Ringhofer <[email protected]>
Tested-by: Csaba Ringhofer <[email protected]>
---
M tests/metadata/test_ddl.py
1 file changed, 14 insertions(+), 31 deletions(-)

Approvals:
  Csaba Ringhofer: Looks good to me, approved; Verified

--
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: merged
Gerrit-Change-Id: I260e40a7cdeae57f18891e009f32a9466b2f8ece
Gerrit-Change-Number: 23308
Gerrit-PatchSet: 5
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