Michael Smith has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22802


Change subject: IMPALA-13631: (Addendum) Retry aborted concurrent DDLs
......................................................................

IMPALA-13631: (Addendum) Retry aborted concurrent DDLs

TestConcurrentDdls has several exceptions it considers acceptable for
testing; it would accept the query failure and continue with other
cases. That was fine for existing queries, but if an ALTER RENAME fails
subsequent queries will also fail because the table does not have the
expected name.

With IMPALA-13631, there are two exception cases we need to handle:
1. "Table/view rename succeeded in the Hive Metastore, but failed in
   Impala's Catalog Server" happens when the HMS alter_table RPC
   succeeds but local catalog has changed. INVALIDATE METADATA on the
   target table is sufficient to bring things in sync.
2. "CatalogException: Table ... was modified while operation was in
   progress, aborting execution" can safely be retried.

The last exception case is removed, as it was not correctly formatted
and would not have matched the exception if it was thrown.

Tested by running test_concurrent_ddls.py 15 times. The 1st exception
previously would show up within 3-4 runs, while the 2nd exception
happens pretty much every run.

Change-Id: I04d071b62e4f306466a69ebd9e134a37d4327b77
---
M tests/custom_cluster/test_concurrent_ddls.py
1 file changed, 30 insertions(+), 24 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/22802/1
--
To view, visit http://gerrit.cloudera.org:8080/22802
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I04d071b62e4f306466a69ebd9e134a37d4327b77
Gerrit-Change-Number: 22802
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com>

Reply via email to