Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9214 )
Change subject: Force inlining of BloomFilter::MakeMask ...................................................................... Force inlining of BloomFilter::MakeMask I noticed that this function was showing up in perf top for TPC-H Q8 running locally. It wasn't inlined into BloomFilter::BucketFindAVX2. Inlining made the query ~5% faster for me locally. Change-Id: I89282f6c315570bea5ad8a0f854cb6eea0592923 Reviewed-on: http://gerrit.cloudera.org:8080/9214 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/benchmarks/bloom-filter-benchmark.cc M be/src/util/bloom-filter.h 2 files changed, 86 insertions(+), 84 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9214 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I89282f6c315570bea5ad8a0f854cb6eea0592923 Gerrit-Change-Number: 9214 Gerrit-PatchSet: 6 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Mostafa Mokhtar <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
