Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11267 )
Change subject: common: add equality methods to ColumnBlock and SelectionVector ...................................................................... Patch Set 9: (4 comments) Sorry for missing this review earlier; I've addressed your comments. http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/columnblock.h File src/kudu/common/columnblock.h: http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/columnblock.h@107 PS2, Line 107: bool Equals(const ColumnBlock& other) const { > see comment in the other header file about operator== Done http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/columnblock.h@114 PS2, Line 114: ^ > nit: as a non-hardware engineer, I have trouble interpreting XOR before noo Done http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/columnblock.h@256 PS2, Line 256: 0 > nit: how about: Done http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/rowblock.h File src/kudu/common/rowblock.h: http://gerrit.cloudera.org:8080/#/c/11267/2/src/kudu/common/rowblock.h@122 PS2, Line 122: bool Equals(const SelectionVector& other) const; > How about operator== ? The style guide recommendation has changed over the Done -- To view, visit http://gerrit.cloudera.org:8080/11267 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9712bd7748bb01af7b6f68897a453a0aa149cdcf Gerrit-Change-Number: 11267 Gerrit-PatchSet: 9 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Wed, 17 Oct 2018 00:33:16 +0000 Gerrit-HasComments: Yes
