Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16613 )

Change subject: IMPALA-10244: Make non-scalable failures to dequeue observable.
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16613/2/be/src/scheduling/admission-controller-test.cc
File be/src/scheduling/admission-controller-test.cc:

http://gerrit.cloudera.org:8080/#/c/16613/2/be/src/scheduling/admission-controller-test.cc@335
PS2, Line 335:       *schedule_state, config_c, true, &not_admitted_reason, 
nullptr, cannot_scale_with_executors));
line too long (100 > 90)


http://gerrit.cloudera.org:8080/#/c/16613/2/be/src/scheduling/admission-controller-test.cc@408
PS2, Line 408:       *schedule_state, config_d, true, &not_admitted_reason, 
nullptr, cannot_scale_with_executors));
line too long (100 > 90)


http://gerrit.cloudera.org:8080/#/c/16613/2/tests/custom_cluster/test_admission_controller.py
File tests/custom_cluster/test_admission_controller.py:

http://gerrit.cloudera.org:8080/#/c/16613/2/tests/custom_cluster/test_admission_controller.py@964
PS2, Line 964: 
flake8: E221 multiple spaces before operator



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3456396ac139c562ad9cd3ac1a624d8f35487518
Gerrit-Change-Number: 16613
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Mon, 19 Oct 2020 21:21:41 +0000
Gerrit-HasComments: Yes

Reply via email to