Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20320 )
Change subject: [consensus] clean-up on OpId utilities ...................................................................... Patch Set 1: (1 comment) Apart from Mahesh's questions rest all looks good to me. http://gerrit.cloudera.org:8080/#/c/20320/1/src/kudu/consensus/opid_util.cc File src/kudu/consensus/opid_util.cc: http://gerrit.cloudera.org:8080/#/c/20320/1/src/kudu/consensus/opid_util.cc@73 PS1, Line 73: 81 > Just curious, why the change from 100 to 81? +1 -- To view, visit http://gerrit.cloudera.org:8080/20320 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iea1e7fba9d91ac9527d88721a867a856e8a858b3 Gerrit-Change-Number: 20320 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Comment-Date: Tue, 08 Aug 2023 17:03:44 +0000 Gerrit-HasComments: Yes
