Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15609 )

Change subject: [ranger] authorize list tables should never throw NotAuthorized
......................................................................

[ranger] authorize list tables should never throw NotAuthorized

When authorizing list tables, the tables the user has no privileges
should be filtered out. However, that should not report an NotAuthorized
error even the user has no privileges over any tables. This also matches
the behavior with Sentry authorization.

Change-Id: I2d1df45a82acc70000783f0cbcce4d3c81840176
Reviewed-on: http://gerrit.cloudera.org:8080/15609
Tested-by: Kudu Jenkins
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, 14 insertions(+), 21 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Andrew Wong: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I2d1df45a82acc70000783f0cbcce4d3c81840176
Gerrit-Change-Number: 15609
Gerrit-PatchSet: 4
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to