Mahesh Reddy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17753 )

Change subject: [common] Update PartitionSchema::operator==()
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/17753/1/src/kudu/common/partition.h
File src/kudu/common/partition.h:

http://gerrit.cloudera.org:8080/#/c/17753/1/src/kudu/common/partition.h@279
PS1, Line 279:   // The format of this string is not guaranteed to be identical 
cross-version.
             :   //
> Is it possible to replace this static method with corresponding operator==(
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I09c626ecdcbe7c0465e6483d6a23369fb7194bfb
Gerrit-Change-Number: 17753
Gerrit-PatchSet: 2
Gerrit-Owner: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Comment-Date: Thu, 05 Aug 2021 02:09:08 +0000
Gerrit-HasComments: Yes

Reply via email to