Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3007: Adjust Bloom Filter size according to NDV estimate ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/2812/1/be/src/exec/old-hash-table.cc File be/src/exec/old-hash-table.cc: Line 159: RawValue::GetHashValue(e, build_expr_ctxs_[i]->root()->type(), oops. this didn't cause tests to fail? http://gerrit.cloudera.org:8080/#/c/2812/1/testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test File testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test: Line 362: join (select * from l LIMIT 2000000) b on a.l_orderkey = -b.l_orderkey; > pick a number that'll exceed the size limit ? http://gerrit.cloudera.org:8080/#/c/2812/1/testdata/workloads/functional-query/queries/QueryTest/runtime_filters_wait.test File testdata/workloads/functional-query/queries/QueryTest/runtime_filters_wait.test: Line 37: row_regex: .*0 of 1 Runtime Filters Produced.* > What would you prefer? In a certain sense, something did go wrong (we guess it sounds like something might still happen. how about "0 filters published, 1 filter disabled"? that way it's clear that this was intentional. -- To view, visit http://gerrit.cloudera.org:8080/2812 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1fe37b8d4cfb3c52bb8e8cf0ca55e92665b87803 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Mostafa Mokhtar <[email protected]> Gerrit-HasComments: Yes
