Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15592 )
Change subject: [ranger] fix authorizing list tables ...................................................................... [ranger] fix authorizing list tables When authorizing list tables and there is no tables in Kudu, we should return 'OK' instead of 'InvalidArgument' error. Change-Id: I2a4dfe61e85fc21d67e6422a1cdb36d887d51f4f Reviewed-on: http://gerrit.cloudera.org:8080/15592 Reviewed-by: Attila Bukor <[email protected]> Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/ranger/ranger_client-test.cc M src/kudu/ranger/ranger_client.cc 2 files changed, 8 insertions(+), 1 deletion(-) Approvals: Attila Bukor: Looks good to me, approved Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15592 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2a4dfe61e85fc21d67e6422a1cdb36d887d51f4f Gerrit-Change-Number: 15592 Gerrit-PatchSet: 2 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)
