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

Change subject: IMPALA-1575: part 2: yield admission control resources
......................................................................


Patch Set 1:

This is essentially the same patch as before, except with the statestore update 
frequency in the test tweaked to avoid the flaky timeout problem we were 
hitting on release builds.

The problem is that queries were sometimes in the queue long enough (60s) to 
hit the default admission control timeout. The reason it was so slow is that 
the test frequently waits for 10 statestore updates or 5 seconds.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib1fae8dc1c4b0eca7bfa8fadae4a56ef2b37947a
Gerrit-Change-Number: 8581
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Fri, 17 Nov 2017 00:57:57 +0000
Gerrit-HasComments: No

Reply via email to