Quanlong Huang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20020
Change subject: IMPALA-12189: updateCatalog should handle failures in createTblTransaction ...................................................................... IMPALA-12189: updateCatalog should handle failures in createTblTransaction updateCatalog() invokes createTblTransaction() for transactional tables. It's called after acquiring the table lock. The write lock of catalog's versionLock will also be acquired by the current thread. Whenever we hit an exception, we should release those locks. This patch moves the code calling createTblTransaction() into the exception handling scope. Tests: - Manually modify the code to throw an TransactionException before calling createTblTransaction. Verified catalogd is hanging after an INSERT. Verified the issue is fixed by this patch. Change-Id: I3a64764d0568fc1e6c6f4c52f9e220df3130bd84 --- M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java 1 file changed, 14 insertions(+), 14 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/20/20020/1 -- To view, visit http://gerrit.cloudera.org:8080/20020 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3a64764d0568fc1e6c6f4c52f9e220df3130bd84 Gerrit-Change-Number: 20020 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]>
