Bikramjeet Vig has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14058 )

Change subject: IMPALA-8791: Handle the case where there is no fragment 
scheduled on the coordinator
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14058/1/tests/custom_cluster/test_admission_controller.py
File tests/custom_cluster/test_admission_controller.py:

http://gerrit.cloudera.org:8080/#/c/14058/1/tests/custom_cluster/test_admission_controller.py@551
PS1, Line 551: # Make sure query execution works perfectly for a query that 
does not have any
             :     # fragments schdeuled on the coordinator, but has 
runtime-filters that need to be
             :     # aggregated at the coordinator.
removed the DCHECK in Coordinator::BackendState::PublishFilter and added this 
test that have triggered that dcheck.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic4fba02bb7b20553a20634f8c5989d43ba2c0721
Gerrit-Change-Number: 14058
Gerrit-PatchSet: 1
Gerrit-Owner: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 13 Aug 2019 22:01:35 +0000
Gerrit-HasComments: Yes

Reply via email to