Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/21632 )
Change subject: [cfile] minor clean-up ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/21632/3/src/kudu/cfile/binary_dict_block.cc File src/kudu/cfile/binary_dict_block.cc: http://gerrit.cloudera.org:8080/#/c/21632/3/src/kudu/cfile/binary_dict_block.cc@a1 PS3, Line 1: > nit: probably a mistype Good catch! Indeed, that's not intended. Fixed now in PS4. -- To view, visit http://gerrit.cloudera.org:8080/21632 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ied096e13b546f29db1ae202f2814b7c5f1529c41 Gerrit-Change-Number: 21632 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com> Gerrit-Reviewer: Marton Greber <greber...@gmail.com> Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com> Gerrit-Comment-Date: Wed, 07 Aug 2024 17:43:02 +0000 Gerrit-HasComments: Yes