Internal Jenkins has submitted this change and it was merged.

Change subject: Runtime filters tests
......................................................................


Runtime filters tests

This patch adds functional tests for runtime filters. It relies on
setting RUNTIME_FILTER_WAIT_TIME_MS high enough to ensure that filters
are received.

To make the test files more readable, this patch also adds a new COMMENT
section to the test syntax, and allows blank spaces between queries so
that the separation of different test cases can be made more obvious.

Currently missing is a test for disabling probe-side filters based on
selectivity, as we lack suitable tables to trigger the disable condition.

Change-Id: I94d617c6d23ffa394a6eb7ead56f1cfb701e0d90
Reviewed-on: http://gerrit.cloudera.org:8080/2603
Reviewed-by: Henry Robinson <[email protected]>
Tested-by: Internal Jenkins
---
A testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test
A 
testdata/workloads/functional-query/queries/QueryTest/runtime_filters_wait.test
A testdata/workloads/functional-query/queries/QueryTest/runtime_row_filters.test
A tests/query_test/test_runtime_filters.py
M tests/util/test_file_parser.py
5 files changed, 637 insertions(+), 3 deletions(-)

Approvals:
  Henry Robinson: Looks good to me, approved
  Internal Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/2603
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I94d617c6d23ffa394a6eb7ead56f1cfb701e0d90
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Henry Robinson <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Internal Jenkins

Reply via email to