Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15635 )

Change subject: rowblock: use BMI instruction set when available for 
GetSelectedRows
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15635/1/src/kudu/common/rowblock.cc
File src/kudu/common/rowblock.cc:

http://gerrit.cloudera.org:8080/#/c/15635/1/src/kudu/common/rowblock.cc@80
PS1, Line 80: // TODO(todd) this is a bit faster when implemented with target 
"bmi" enabled.
            : // Consider duplicating it and doing runtime switching.
nit: can this be removed now?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ec74bc5db07c18d0e36de14a2343f49fc5c2859
Gerrit-Change-Number: 15635
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 01 Apr 2020 23:20:04 +0000
Gerrit-HasComments: Yes

Reply via email to