DRILL-4592: Explain plan statement should show plan in WebUI
Project: http://git-wip-us.apache.org/repos/asf/drill/repo Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/9f4fff80 Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/9f4fff80 Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/9f4fff80 Branch: refs/heads/master Commit: 9f4fff800d128878094ae70b454201f79976135d Parents: dbf4b15 Author: Jinfeng Ni <[email protected]> Authored: Thu Apr 7 11:24:04 2016 -0700 Committer: Jinfeng Ni <[email protected]> Committed: Mon Apr 11 16:23:48 2016 -0700 ---------------------------------------------------------------------- .../java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java | 2 +- .../apache/drill/exec/planner/sql/handlers/ExplainHandler.java | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/drill/blob/9f4fff80/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java index 60daac1..dbe620d 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java @@ -70,7 +70,7 @@ public class DrillSqlWorker { switch(sqlNode.getKind()){ case EXPLAIN: - handler = new ExplainHandler(config); + handler = new ExplainHandler(config, textPlan); break; case SET_OPTION: handler = new SetOptionHandler(context); http://git-wip-us.apache.org/repos/asf/drill/blob/9f4fff80/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ExplainHandler.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ExplainHandler.java b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ExplainHandler.java index 62e44d8..b5b5f73 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ExplainHandler.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ExplainHandler.java @@ -39,6 +39,7 @@ import org.apache.drill.exec.planner.logical.DrillRel; import org.apache.drill.exec.planner.physical.Prel; import org.apache.drill.exec.planner.physical.explain.PrelSequencer; import org.apache.drill.exec.planner.sql.DirectPlan; +import org.apache.drill.exec.util.Pointer; import org.apache.drill.exec.work.foreman.ForemanSetupException; public class ExplainHandler extends DefaultSqlHandler { @@ -46,8 +47,8 @@ public class ExplainHandler extends DefaultSqlHandler { private ResultMode mode; private SqlExplainLevel level = SqlExplainLevel.ALL_ATTRIBUTES; - public ExplainHandler(SqlHandlerConfig config) { - super(config); + public ExplainHandler(SqlHandlerConfig config, Pointer<String> textPlan) { + super(config, textPlan); } @Override
