Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15696 )
Change subject: [ranger] Fix and refactor RangerClient ...................................................................... [ranger] Fix and refactor RangerClient While working on integration tests we found a discrepancy handling column-level privileges between Ranger and Sentry. This was caused by RangerClient returning NotAuthorized when the requested action couldn't be authorized on any of the resources. This commit moves all authorization logic to the authorization provider instead of the client. Change-Id: I3fb7272601c9cb7ebe2e250bea728e76894d242a Reviewed-on: http://gerrit.cloudera.org:8080/15696 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Attila Bukor <[email protected]> Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/master/ranger_authz_provider.cc M src/kudu/ranger/ranger_client-test.cc M src/kudu/ranger/ranger_client.cc M src/kudu/ranger/ranger_client.h 4 files changed, 219 insertions(+), 166 deletions(-) Approvals: Alexey Serbin: Looks good to me, but someone else must approve Attila Bukor: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15696 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3fb7272601c9cb7ebe2e250bea728e76894d242a Gerrit-Change-Number: 15696 Gerrit-PatchSet: 6 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
