Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11734
Change subject: IMPALA-7729: Fix invalidate metadata hang when there is an upper case role name ...................................................................... IMPALA-7729: Fix invalidate metadata hang when there is an upper case role name Sentry stores the role names in lower case and Impala stores the role names based on the original input role names. IMPALA-7343 introduced a new bulk API (listAllRolesPrivileges) from Sentry that returns a map of role name to a set of privileges. Since Impala preserves the case sensitivity of the role names based on the original input role names, this causes an issue when trying to retrieve a set of privileges from a role name that is stored in Impala, especially when the role names in Impala differ than the ones returned by listAllRolesPrivileges. This issue will then result in privileges with mismatch role names to never get refreshed in the Catalogd, which causes Impalad to wait indefinitely waiting for the privileges to be updated by Catalogd. The fix is to get a set of privileges using the role names returned by Sentry's listAllRoles instead of using the role names stored in Impala. Testing: - Added a new E2E test - Ran all E2E authorization tests Change-Id: I5aa6f626ad3df4e9321ed18273d045517bc099c2 --- M fe/src/main/java/org/apache/impala/util/SentryProxy.java M tests/authorization/test_grant_revoke.py 2 files changed, 29 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/34/11734/4 -- To view, visit http://gerrit.cloudera.org:8080/11734 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I5aa6f626ad3df4e9321ed18273d045517bc099c2 Gerrit-Change-Number: 11734 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com>
