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

Change subject: KUDU-16 pt 1: add server-side limits on scanners
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9783/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/9783/1//COMMIT_MSG@19
PS1, Line 19:
            : Change-Id: I3ca5dd
> I'd like to avoid using unsigned types for non-bit patterns, unless you mea
I don't really care about unsigned vs. signed, but I thought it'd be nice to 
reuse the existing proto field. I don't think doing so will have any effect on 
backwards compatibility.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3ca5ddff09f4910d12f80259bd11e4027f99aa7c
Gerrit-Change-Number: 9783
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Fri, 23 Mar 2018 20:11:13 +0000
Gerrit-HasComments: Yes

Reply via email to