Fucun Chu has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17744


Change subject: IMPALA-10835: Extend the DS_HLL_SKETCH function to accept a 
precision
......................................................................

IMPALA-10835: Extend the DS_HLL_SKETCH function to accept a precision

This path addresses the current limitation in DS_HLL_SKETCH function by
extending the function to optionally take a secondary argument
called precision.

   DS_HLL_SKETCH(expression [, precision])

The precision value must be between 4 and 21, specified as an integer
literal. The default is 12.

Here are test results of a typical workload in tpch25.lineitem (#1):
+====================================================================+
|   Metric    | Count Distinct | DS_HLL-12  | DS_HLL-16  | DS_HLL-21 |
+--------------------------------------------------------------------+
|  Memory(MB) |     725.43     |   124.87   |    123.19  |    121.85 |
| Duration(s) |      5.64      |   1.03     |    1.13    |     1.64  |
|  ErrorRate  |       0%       |   1.26%    |    0.22%   |     0.05% |
+====================================================================+

Testing:
1. Ran unit tests against table lineitem in TPC-DS in both serial and
   parallel plan settings;
2. Ran "core" tests.

Change-Id: I91a360bb046d4abb101641772b6159308bf6c014
---
M be/src/exprs/aggregate-functions-ir.cc
M be/src/exprs/aggregate-functions.h
M be/src/exprs/datasketches-common.h
M fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java
M fe/src/main/java/org/apache/impala/catalog/BuiltinsDb.java
M testdata/workloads/functional-query/queries/QueryTest/datasketches-hll.test
6 files changed, 155 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/44/17744/1
--
To view, visit http://gerrit.cloudera.org:8080/17744
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I91a360bb046d4abb101641772b6159308bf6c014
Gerrit-Change-Number: 17744
Gerrit-PatchSet: 1
Gerrit-Owner: Fucun Chu <chufu...@hotmail.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Gabor Kaszab <gaborkas...@cloudera.com>

Reply via email to