Michael Smith has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/24720 Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/service/internal-server-test.cc 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Michael Smith: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I6b960aecb4bf50395fc64ed8263b02b6b92180d2 Gerrit-Change-Number: 24720 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
