Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/24720


Change subject: IMPALA-15265: Fix internal-server-test's long execution time
......................................................................

IMPALA-15265: Fix internal-server-test's long execution time

After the default file format changed to Parquet, the
internal-server-test's QueryTimeout has been taking
35+ minutes to run. The test runs a SQL with a predicate
"where id=sleep(2000000)" and FETCH_ROWS_TIMEOUT_MS=1.
The idea is that the sleep happens during execution and
we hit the timeout. With the switch to Parquet, the
logic for Parquet dictionary filtering is evaluating
the predicate at planning time to determine if it
returns true for NULL. This causes it to sleep for
35+ minutes, and the FETCH_ROWS_TIMEOUT_MS doesn't
kick in.

This turns off Parquet dictionary filtering for this
test, which avoids the sleep during planning. It also
adds a check of the time that the test runs to verify
that it runs in less than a minute.

Testing:
 - Ran test locally

Change-Id: I6b960aecb4bf50395fc64ed8263b02b6b92180d2
---
M be/src/service/internal-server-test.cc
1 file changed, 12 insertions(+), 1 deletion(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b960aecb4bf50395fc64ed8263b02b6b92180d2
Gerrit-Change-Number: 24720
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>

Reply via email to