Hello anujphadke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9078
to look at the new patch set (#2).
Change subject: IMPALA-6422: Use ldexp() instead of powf() in HLL.
......................................................................
IMPALA-6422: Use ldexp() instead of powf() in HLL.
Using ldexp() to compute a floating point power of two is
over 10x faster than powf().
This change is particularly helpful for speeding up
COMPUTE STATS TABLESAMPLE which has many calls to
HllFinalEstimate() where floating point power of two
computations are relevant.
Testing:
- core/hdfs run passed
Change-Id: I517614d3f9cf1cf56b15a173c3cfe76e0f2e0382
---
M be/src/exprs/aggregate-functions-ir.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/9078/2
--
To view, visit http://gerrit.cloudera.org:8080/9078
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I517614d3f9cf1cf56b15a173c3cfe76e0f2e0382
Gerrit-Change-Number: 9078
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Behm <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: anujphadke <[email protected]>
