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