Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 )
Change subject: WIP KUDU-2543: pass around default authz tokens ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11751/1/src/kudu/master/master_service.cc File src/kudu/master/master_service.cc: http://gerrit.cloudera.org:8080/#/c/11751/1/src/kudu/master/master_service.cc@416 PS1, Line 416: TablePrivilegePB table_privilege; : table_privilege.set_table_id(resp->table_id()); : table_privilege.set_all_privileges(true); As implemented, this is just mocking any calls to an AuthzProvider that would, e.g. make a call to Sentry and translate the Sentry privileges into the TablePrivilegePB format, or e.g. check that the connected user is a trusted user and assign all privileges. -- To view, visit http://gerrit.cloudera.org:8080/11751 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I99555e0ab2d09d4abcbc12b1100658a9a17590f4 Gerrit-Change-Number: 11751 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Mon, 22 Oct 2018 23:12:25 +0000 Gerrit-HasComments: Yes