Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10602 )

Change subject: IMPALA-6956: deflake and logging for query_expiration test
......................................................................


Patch Set 1:

(1 comment)

Thanks for looking at this, sorry for creating the problem in the first place.

http://gerrit.cloudera.org:8080/#/c/10602/1/tests/custom_cluster/test_query_expiration.py
File tests/custom_cluster/test_query_expiration.py:

http://gerrit.cloudera.org:8080/#/c/10602/1/tests/custom_cluster/test_query_expiration.py@58
PS1, Line 58:     client.execute("SET EXEC_TIME_LIMIT_S=3")
I understand this fixes one of the problem - this query expires too soon, but I 
don't see how this change fixes the query that was slow to start. Should we be 
waiting for all four queries to getting into the RUNNING state?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I01a8762d28ad920b9ec8a0b1b82469618c66768f
Gerrit-Change-Number: 10602
Gerrit-PatchSet: 1
Gerrit-Owner: Vuk Ercegovac <vercego...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Tue, 05 Jun 2018 03:24:59 +0000
Gerrit-HasComments: Yes

Reply via email to