Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15767 )
Change subject: [util] Don't use static function ptrs for non-static member functions in BlockBloom filter ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/15767/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15767/1//COMMIT_MSG@25 PS1, Line 25: - Uses non-static member function pointers for Insert/Find that are initialized : in constructor(this is same as the change before introduction of OrEqual functions) : avoiding the branch in Insert/Find. : - Doesn't use a static function pointer for OrEqual functions and checks for AVX2 : capability at run-time on every invocation. > Ah I see. In that case, could you update the preface to these bullets? The Done http://gerrit.cloudera.org:8080/#/c/15767/1/src/kudu/util/block_bloom_filter.h File src/kudu/util/block_bloom_filter.h: http://gerrit.cloudera.org:8080/#/c/15767/1/src/kudu/util/block_bloom_filter.h@270 PS1, Line 270: to > Ah, seems I misinterpreted this. I think mentioning static/non-static here Done -- To view, visit http://gerrit.cloudera.org:8080/15767 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I39703fd1a7e256ff60ef86d0b370590fbb526380 Gerrit-Change-Number: 15767 Gerrit-PatchSet: 1 Gerrit-Owner: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Comment-Date: Fri, 24 Apr 2020 01:40:35 +0000 Gerrit-HasComments: Yes
