Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/14103 )
Change subject: IMPALA-8858: Add metrics tracking num queries running on executor groups ...................................................................... Patch Set 16: Code-Review+1 (2 comments) carrying over Lars's +1 http://gerrit.cloudera.org:8080/#/c/14103/15/tests/custom_cluster/test_executor_groups.py File tests/custom_cluster/test_executor_groups.py: http://gerrit.cloudera.org:8080/#/c/14103/15/tests/custom_cluster/test_executor_groups.py@90 PS15, Line 90: > flake8: E712 comparison to True should be 'if cond is True:' or 'if cond:' Done http://gerrit.cloudera.org:8080/#/c/14103/15/tests/custom_cluster/test_executor_groups.py@100 PS15, Line 100: > flake8: E712 comparison to True should be 'if cond is True:' or 'if cond:' Done -- To view, visit http://gerrit.cloudera.org:8080/14103 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I58cde8699c33af8b87273437e9d8bf6371a34539 Gerrit-Change-Number: 14103 Gerrit-PatchSet: 16 Gerrit-Owner: Bikramjeet Vig <bikramjeet....@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-Comment-Date: Tue, 17 Sep 2019 18:19:28 +0000 Gerrit-HasComments: Yes
