impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Internal Jenkins (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4020: Handle external conflicting changes to HMS gracefully
Date Thu, 01 Sep 2016 04:15:05 GMT
Internal Jenkins has submitted this change and it was merged.

Change subject: IMPALA-4020: Handle external conflicting changes to HMS gracefully
......................................................................


IMPALA-4020: Handle external conflicting changes to HMS gracefully

Currently Catalog can't handle conflicting changes to HMS' databases
from external clients while running invalidate metadata operation.
For example, if a database is dropped by a client external to Impala,
while the invalidate metadata is in process, Catalog aborts the
metadata load. This commit fixes this issue by handling appropriate
exceptions when HMS operations fail and only ignores the load for that
particular database.

Change-Id: Ic228efbcceb9ef6c165d0d9aeef7202581e3e46a
Reviewed-on: http://gerrit.cloudera.org:8080/4161
Reviewed-by: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Reviewed-by: Marcel Kornacker <marcel@cloudera.com>
Tested-by: Internal Jenkins
---
M fe/src/main/java/com/cloudera/impala/catalog/CatalogServiceCatalog.java
1 file changed, 61 insertions(+), 31 deletions(-)

Approvals:
  Marcel Kornacker: Looks good to me, approved
  Internal Jenkins: Verified
  Dimitris Tsirogiannis: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.cloudera.org:8080/4161
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic228efbcceb9ef6c165d0d9aeef7202581e3e46a
Gerrit-PatchSet: 7
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: anujphadke <aphadke@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bharathv@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Marcel Kornacker <marcel@cloudera.com>
Gerrit-Reviewer: anujphadke <aphadke@cloudera.com>

Mime
View raw message