Wenzhe Zhou has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16094
Change subject: IMPALA-9691: Support Kudu Timestamp and Date bloom filter ...................................................................... IMPALA-9691: Support Kudu Timestamp and Date bloom filter Impala save timestamp as 12 bytes of structure TimestampValue with time in nano seconds. Kudu store timestamp as 8 bytes of Unix Time microseconds. To avoid the data truncation issue in the bloom filter, define TimestampTruncationExpr expression, and use it as the root of source expression of bloom filter to convert timestamp as microseconds when building bloom filter for Kudu. Generated functional date_tbl table in Kudu format for unit-test. Added new test cases for Kudu Timestamp and Date bloom filters. Testing: Passed test_runtime_filters.py, test_kudu.py and Planner test. Change-Id: I3c1e9bcc9fd6d79a39f25eaa3396188fc0a52a48 --- M be/src/exprs/CMakeLists.txt M be/src/exprs/scalar-expr.cc M be/src/exprs/scalar-expr.h A be/src/exprs/timestamp-truncation-expr.cc A be/src/exprs/timestamp-truncation-expr.h M common/thrift/Exprs.thrift A fe/src/main/java/org/apache/impala/analysis/TimestampTruncationExpr.java M fe/src/main/java/org/apache/impala/planner/RuntimeFilterGenerator.java M testdata/datasets/functional/functional_schema_template.sql M testdata/datasets/functional/schema_constraints.csv M testdata/workloads/functional-planner/queries/PlannerTest/runtime-filter-propagation.test M testdata/workloads/functional-query/queries/QueryTest/all_runtime_filters.test 12 files changed, 363 insertions(+), 23 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/16094/1 -- To view, visit http://gerrit.cloudera.org:8080/16094 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3c1e9bcc9fd6d79a39f25eaa3396188fc0a52a48 Gerrit-Change-Number: 16094 Gerrit-PatchSet: 1 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
