[
https://issues.apache.org/jira/browse/DRILL-4465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15178096#comment-15178096
]
ASF GitHub Bot commented on DRILL-4465:
---------------------------------------
Github user jinfengni commented on a diff in the pull request:
https://github.com/apache/drill/pull/401#discussion_r54908424
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/SqlHandlerConfig.java
---
@@ -18,30 +18,41 @@
package org.apache.drill.exec.planner.sql.handlers;
-import org.apache.calcite.tools.Planner;
+import java.util.Collection;
+import java.util.Map.Entry;
+
+import org.apache.calcite.tools.RuleSet;
import org.apache.drill.exec.ops.QueryContext;
-import org.apache.calcite.plan.hep.HepPlanner;
+import org.apache.drill.exec.planner.PlannerPhase;
+import org.apache.drill.exec.planner.sql.DrillSqlParser;
+import org.apache.drill.exec.store.StoragePlugin;
+
+import com.beust.jcommander.internal.Lists;
--- End diff --
Import a wrong "Lists" here? checkstyle complained this import.
> Refactor Parsing and Planning to canonicalize planning and parsing
> ------------------------------------------------------------------
>
> Key: DRILL-4465
> URL: https://issues.apache.org/jira/browse/DRILL-4465
> Project: Apache Drill
> Issue Type: Sub-task
> Components: Query Planning & Optimization
> Reporter: Jacques Nadeau
> Assignee: Jinfeng Ni
> Fix For: 1.6.0
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
