Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13681 )

Change subject: KUDU-2870: allow super-user to skip authz checks in Checksum
......................................................................


Patch Set 6:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13681/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/13681/5//COMMIT_MSG@15
PS5, Line 15: control.
> nit: maybe, rephrase a bit for clarity:
Updated


http://gerrit.cloudera.org:8080/#/c/13681/5/src/kudu/tserver/tablet_server_authorization-test.cc
File src/kudu/tserver/tablet_server_authorization-test.cc:

PS5:
> nit: maybe, remove the changes in this file since they are not essential an
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9da21f41702da747a081ab037d75865748d981a8
Gerrit-Change-Number: 13681
Gerrit-PatchSet: 6
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Thu, 20 Jun 2019 07:21:33 +0000
Gerrit-HasComments: Yes

Reply via email to