You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by sa...@apache.org on 2018/04/05 02:58:23 UTC
[3/5] impala git commit: IMPALA-6771: Fix in-predicate set up bug
IMPALA-6771: Fix in-predicate set up bug
Fixes a bug that introduced default initialized values in the set data
structure used to check for set membership that can cause wrong results.
Testing:
Added a test case that checks for the same.
Change-Id: I7e776dbcb7ee4a9b64e1295134a27d332f5415b6
Reviewed-on: http://gerrit.cloudera.org:8080/9891
Reviewed-by: Sailesh Mukil <sa...@cloudera.com>
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Tested-by: Impala Public Jenkins
Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/75e1bd1b
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/75e1bd1b
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/75e1bd1b
Branch: refs/heads/master
Commit: 75e1bd1bcd001770511247f2a762d9e74f38ce4f
Parents: b80a75f
Author: Bikramjeet Vig <bi...@cloudera.com>
Authored: Mon Apr 2 13:55:48 2018 -0700
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Wed Apr 4 21:51:29 2018 +0000
----------------------------------------------------------------------
be/src/exprs/in-predicate.h | 3 ++-
.../functional-query/queries/QueryTest/exprs.test | 12 ++++++++++++
2 files changed, 14 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/impala/blob/75e1bd1b/be/src/exprs/in-predicate.h
----------------------------------------------------------------------
diff --git a/be/src/exprs/in-predicate.h b/be/src/exprs/in-predicate.h
index 2d439ba..87ae8b8 100644
--- a/be/src/exprs/in-predicate.h
+++ b/be/src/exprs/in-predicate.h
@@ -350,7 +350,8 @@ void InPredicate::SetLookupPrepare(
state->contains_null = false;
// Collect all values in a vector to use the bulk insert API to avoid N^2 behavior
// with flat_set.
- vector<SetType> element_list(ctx->GetNumArgs());
+ std::vector<SetType> element_list;
+ element_list.reserve(ctx->GetNumArgs() - 1);
for (int i = 1; i < ctx->GetNumArgs(); ++i) {
DCHECK(ctx->IsArgConstant(i));
T* arg = reinterpret_cast<T*>(ctx->GetConstantArg(i));
http://git-wip-us.apache.org/repos/asf/impala/blob/75e1bd1b/testdata/workloads/functional-query/queries/QueryTest/exprs.test
----------------------------------------------------------------------
diff --git a/testdata/workloads/functional-query/queries/QueryTest/exprs.test b/testdata/workloads/functional-query/queries/QueryTest/exprs.test
index 57ff3c6..1ac96f0 100644
--- a/testdata/workloads/functional-query/queries/QueryTest/exprs.test
+++ b/testdata/workloads/functional-query/queries/QueryTest/exprs.test
@@ -2995,3 +2995,15 @@ select cast('2001-1-2' as timestamp)
---- TYPES
timestamp
====
+---- QUERY
+# IMPALA-6771: Test that the in-predicate set does not have default initialized
+# values that can result in wrong results. For a string column the default initialized
+# value is an empty string.
+select count(*) from functional.alltypes
+where regexp_replace(string_col, '1', '')
+in ('0', '1', '2', '3', '4', '5', '6', '7', '8', '9')
+---- RESULTS
+6570
+---- TYPES
+bigint
+====