anujphadke has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case ......................................................................
Patch Set 3: Discussed this issue with MJ. This issue does occur due to inconsistent handling of casing. Changed buildRolePrivilegeName to return the privilege names in lower case instead of changing it in privilegeSpec. Also changed getRolePrivileges to return the output of show grant roles in lower case since we are not lower casing privilegeSpec object anymore. Added a test in test_grant_revoke.py and made sentryProxy configurable. -- To view, visit http://gerrit.cloudera.org:8080/7332 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ide3dfa601fcf77f5acc6adce9bea443aea600901 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: anujphadke <apha...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: anujphadke <apha...@cloudera.com> Gerrit-HasComments: No
