Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12436 )

Change subject: IMPALA-8182: Add single-node plan to PlanCtx
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12436/2/fe/src/main/java/org/apache/impala/planner/Planner.java
File fe/src/main/java/org/apache/impala/planner/Planner.java:

http://gerrit.cloudera.org:8080/#/c/12436/2/fe/src/main/java/org/apache/impala/planner/Planner.java@89
PS2, Line 89:     public List<PlanFragment> finalPlan() { return finalPlan_; }
this is not the final plan when mt_dop is set? (createParallelPlan())?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I983473f259df8af17d8dd38e497fe33e839cd0dc
Gerrit-Change-Number: 12436
Gerrit-PatchSet: 2
Gerrit-Owner: Paul Rogers <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Paul Rogers <[email protected]>
Gerrit-Comment-Date: Thu, 14 Feb 2019 00:50:38 +0000
Gerrit-HasComments: Yes

Reply via email to