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 (#2).
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. Combined they give concurrent ALTERs 5 seconds of
additional overhead for processing each of them, beyond the debug
action.
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, 34 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/04/22804/2
--
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: 2
Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>
