drill-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From j..@apache.org
Subject [2/2] drill git commit: DRILL-4592: Explain plan statement should show plan in WebUI
Date Wed, 13 Apr 2016 00:23:49 GMT
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 <jni@apache.org>
Authored: Thu Apr 7 11:24:04 2016 -0700
Committer: Jinfeng Ni <jni@apache.org>
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


Mime
View raw message