Mahesh Reddy has posted comments on this change. (
http://gerrit.cloudera.org:8080/18640 )
Change subject: KUDU-2671 update Add{ExclusiveUpper,Lower}BoundPartitionKeyRaw
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/18640/1/src/kudu/client/client.cc
File src/kudu/client/client.cc:
http://gerrit.cloudera.org:8080/#/c/18640/1/src/kudu/client/client.cc@1786
PS1, Line 1786: // Whatever range partition the 'partition_key' actually
belongs to and
nit: Maybe could rephrase the first sentence to say something along the lines
of "the range partition and respective hash schema used to encode the key are
irrelevant here as the table wide hash schema is used as a proxy..." rather
than saying "whatever", seems a bit informal.
--
To view, visit http://gerrit.cloudera.org:8080/18640
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1323e494733c9c082dedc12d941811f4175dbeab
Gerrit-Change-Number: 18640
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Comment-Date: Mon, 20 Jun 2022 19:18:18 +0000
Gerrit-HasComments: Yes
