impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Impala Public Jenkins (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5571: Fix authorization tests
Date Tue, 27 Jun 2017 07:39:45 GMT
Impala Public Jenkins has submitted this change and it was merged.

Change subject: IMPALA-5571: Fix authorization tests
......................................................................


IMPALA-5571: Fix authorization tests

Issue:
With the fix for IMPALA-5549 the catalog cache of an impalad may end up
with stale authorization metadata in the presence of GRANT/REVOKE
statements that modify multiple privileges.

Fix:
Revert the behavior of GRANT/REVOKE privilege DDL statements.

Change-Id: Icad893a10fdeb7a721264e69413014603dec6339
Reviewed-on: http://gerrit.cloudera.org:8080/7299
Reviewed-by: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Tested-by: Impala Public Jenkins
---
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Impala Public Jenkins: Verified
  Dimitris Tsirogiannis: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icad893a10fdeb7a721264e69413014603dec6339
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Dimitris Tsirogiannis <dtsirogiannis@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: Impala Public Jenkins

Mime
View raw message