Add JSON to WriterTarget

Author: tanjialiang

presto-main-base/src/main/java/com/facebook/presto/sql/planner/CanonicalPlan.java

@@ -13,6 +13,7 @@
  */
 package com.facebook.presto.sql.planner;
 
+import com.facebook.airlift.log.Logger;
 import com.facebook.presto.common.plan.PlanCanonicalizationStrategy;
 import com.facebook.presto.spi.PrestoException;
 import com.facebook.presto.spi.plan.PlanNode;
@@ -29,6 +30,7 @@
 
 public class CanonicalPlan
 {
+    private static final Logger log = Logger.get(CanonicalPlan.class);
     private final PlanNode plan;
     private final PlanCanonicalizationStrategy strategy;
 

presto-main-base/src/main/java/com/facebook/presto/sql/planner/CanonicalPlanGenerator.java

@@ -66,6 +66,7 @@
 import com.facebook.presto.sql.planner.plan.SequenceNode;
 import com.facebook.presto.sql.planner.plan.TopNRowNumberNode;
 import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
@@ -1081,7 +1082,7 @@ private Optional<List<Integer>> orderSourcesByTables(List<PlanNode> sources)
         return Optional.of(sourceToPosition.values().stream().collect(toImmutableList()));
     }
 
-    private static class CanonicalWriterTarget
+    public static class CanonicalWriterTarget
             extends TableWriterNode.WriterTarget
     {
         private final ConnectorId connectorId;
@@ -1110,13 +1111,15 @@ public String getWriterTargetType()
         }
 
         @Override
+        @JsonIgnore
         public SchemaTableName getSchemaTableName()
         {
             // Just return a sample table name, which is always same
             return new SchemaTableName("schema", "table");
         }
 
         @Override
+        @JsonIgnore
         public Optional<List<OutputColumnMetadata>> getOutputColumns()
         {
             return Optional.empty();

presto-spi/pom.xml

@@ -102,13 +102,13 @@
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-databind</artifactId>
-            <scope>test</scope>
+            <scope>provided</scope>
         </dependency>
 
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-core</artifactId>
-            <scope>test</scope>
+            <scope>provided</scope>
         </dependency>
 
         <dependency>
@@ -146,5 +146,11 @@
             <artifactId>commons-math3</artifactId>
             <scope>test</scope>
         </dependency>
+
+        <dependency>
+            <groupId>javax.inject</groupId>
+            <artifactId>javax.inject</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 </project>

presto-spi/src/main/java/com/facebook/presto/spi/ColumnMetadata.java

@@ -89,6 +89,7 @@ public Map<String, Object> getProperties()
     {
         return properties;
     }
+
     public Builder toBuilder()
     {
         return ColumnMetadata.builder()