Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15808 )

Change subject: [partitioning] KUDU-2671 [part 2] Added RangeBoundsPB protobuf 
message
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/15808/1/src/kudu/master/master.proto
File src/kudu/master/master.proto:

http://gerrit.cloudera.org:8080/#/c/15808/1/src/kudu/master/master.proto@482
PS1, Line 482: {
style nit: move the bracket to the line above like it's in other message 
definitions in this file.


http://gerrit.cloudera.org:8080/#/c/15808/1/src/kudu/master/master.proto@484
PS1, Line 484:
style nit: shift here should be 2 spaces


http://gerrit.cloudera.org:8080/#/c/15808/1/src/kudu/master/master.proto@500
PS1, Line 500: RangeBoundsPB
Is this change backwards-compatible?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic407d54970475fb1522b3801f62341d0ff1ccd2c
Gerrit-Change-Number: 15808
Gerrit-PatchSet: 1
Gerrit-Owner: Volodymyr Verovkin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 29 Apr 2020 06:25:30 +0000
Gerrit-HasComments: Yes

Reply via email to