Hello Quanlong Huang, Riza Suminto, Sai Hemanth Gantasala, Joe McDonnell, Csaba 
Ringhofer, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-13631: (Addendum) Test slow concurrent alters
......................................................................

IMPALA-13631: (Addendum) Test slow concurrent alters

Adds a test that multiple slow concurrent alters complete in parallel
rather than being executed sequentially.

The test creates tables, and ensures Impala's catalog is up-to-date for
their creation so that starting ALTER TABLE will be fast. Then starts
two ALTER TABLE RENAMES asynchronously - with debug_action ensuring each
takes at least 5 seconds - and waits for them to finish.

Timeouts are chosen so that there's a high likelihood one would fail if
ALTERs were serialized, while providing sufficient time for a busy
system to respond. The 1st timeout is 10 seconds, which prior to
IMPALA-13631 would sometimes fail if the 2nd ALTER started first. The
2nd timeout ensures the other ALTER finishes less than 5 seconds after
the first, which is only possible if they execute the debug action
concurrently.

Also verifies that concurrent alters are no longer blocked on "Got
catalog version read lock".

Change-Id: I87d077aaa295943a16e6da60a2755dd289f3a132
---
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/main/java/org/apache/impala/util/DebugUtils.java
M tests/metadata/test_ddl.py
3 files changed, 47 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/04/22804/3
--
To view, visit http://gerrit.cloudera.org:8080/22804
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I87d077aaa295943a16e6da60a2755dd289f3a132
Gerrit-Change-Number: 22804
Gerrit-PatchSet: 3
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>

Reply via email to