Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13942 )
Change subject: IMPALA-8451: enable admission control for dockerised tests ...................................................................... Patch Set 15: (2 comments) http://gerrit.cloudera.org:8080/#/c/13942/15/bin/jenkins/dockerized-impala-run-tests.sh File bin/jenkins/dockerized-impala-run-tests.sh: http://gerrit.cloudera.org:8080/#/c/13942/15/bin/jenkins/dockerized-impala-run-tests.sh@83 PS15, Line 83: TEST_START_CLUSTER_ARGS+=" --impalad_args=-fair_scheduler_allocation_path=${FAIR_SCHED_CONF}" line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/13942/15/tests/webserver/test_web_pages.py File tests/webserver/test_web_pages.py: http://gerrit.cloudera.org:8080/#/c/13942/15/tests/webserver/test_web_pages.py@544 PS15, Line 544: d flake8: F841 local variable 'default_pool_json' is assigned to but never used -- To view, visit http://gerrit.cloudera.org:8080/13942 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7517673f9e348780fcf7cd6ce1f12c9c5a55373a Gerrit-Change-Number: 13942 Gerrit-PatchSet: 15 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Tue, 17 Sep 2019 00:55:39 +0000 Gerrit-HasComments: Yes