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

Change subject: IMPALA-12312: Using correct executor group set info for planning
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/20273/1/fe/src/main/java/org/apache/impala/service/Frontend.java
File fe/src/main/java/org/apache/impala/service/Frontend.java:

http://gerrit.cloudera.org:8080/#/c/20273/1/fe/src/main/java/org/apache/impala/service/Frontend.java@1972
PS1, Line 1972:       // If the default group exists, fall back to the default 
group.
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/20273/1/tests/custom_cluster/test_executor_groups.py
File tests/custom_cluster/test_executor_groups.py:

http://gerrit.cloudera.org:8080/#/c/20273/1/tests/custom_cluster/test_executor_groups.py@1468
PS1, Line 1468:
flake8: W291 trailing whitespace


http://gerrit.cloudera.org:8080/#/c/20273/1/tests/custom_cluster/test_executor_groups.py@1468
PS1, Line 1468:     # Planner assumes that the query run on the default group
line has trailing whitespace



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia13fb40558441d4dcc0b3e7910d3746bb61e6b80
Gerrit-Change-Number: 20273
Gerrit-PatchSet: 1
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Thu, 27 Jul 2023 13:11:10 +0000
Gerrit-HasComments: Yes

Reply via email to