Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15206 )

Change subject: KUDU-2972 Add Ranger client
......................................................................


Patch Set 12:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15206/12/src/kudu/ranger/ranger_action.h
File src/kudu/ranger/ranger_action.h:

http://gerrit.cloudera.org:8080/#/c/15206/12/src/kudu/ranger/ranger_action.h@29
PS12, Line 29:   INSERT,
             :   UPDATE,
             :   DELETE,
             :   ALTER,
             :   CREATE,
             :   DROP,
Drawing from CRUD for a minute, I'm curious why we chose to separate offer 
"create", "update", and "delete" actions for DDL and DML operations. Why not 
use the same set of actions for both, but require broader scoping for DDL (i.e. 
ON DATABASE vs. ON TABLE)?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie2e1ec19ed3aeb4d82ad38fe1fb655f57021c1a4
Gerrit-Change-Number: 15206
Gerrit-PatchSet: 12
Gerrit-Owner: Attila Bukor <[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)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Wed, 26 Feb 2020 21:33:01 +0000
Gerrit-HasComments: Yes

Reply via email to