[GitHub] [hive] miklosgergely commented on a change in pull request #1756: HIVE-24509 Move show specific codes under DDL and cut MetaDataFormatter classes to pieces

[GitBox]

miklosgergely commented on a change in pull request #1756:
URL: https://github.com/apache/hive/pull/1756#discussion_r552905387



##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/lock/show/ShowDbLocksAnalyzer.java
##########
@@ -23,9 +23,6 @@
 import org.apache.hadoop.hive.ql.ddl.DDLWork;
 import org.apache.hadoop.hive.ql.exec.Task;
 import org.apache.hadoop.hive.ql.exec.TaskFactory;
-import org.apache.hadoop.hive.ql.lockmgr.HiveTxnManager;
-import org.apache.hadoop.hive.ql.lockmgr.LockException;
-import org.apache.hadoop.hive.ql.lockmgr.TxnManagerFactory;

Review comment:
       Removed.

##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/partition/add/AlterTableAddPartitionDesc.java
##########
@@ -234,7 +233,7 @@ public void setWriteId(long writeId) {
 
   @Override
   public String getFullTableName() {
-    return AcidUtils.getFullTableName(dbName,tableName);
+    return AcidUtils.getFullTableName(dbName, tableName);

Review comment:
       Removed.

##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/skewed/AlterTableSetSkewedLocationAnalyzer.java
##########
@@ -20,7 +20,6 @@
 
 import java.net.URI;
 import java.net.URISyntaxException;
-import java.util.ArrayList;

Review comment:
       Removed.

##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/ddl/view/create/AlterViewAsAnalyzer.java
##########
@@ -84,7 +84,7 @@ private void validateCreateView(AlterViewAsDesc desc, 
SemanticAnalyzer analyzer)
 
     if (oldView == null) {
       String viewNotExistErrorMsg = "The following view does not exist: " + 
desc.getViewName();
-      throw new SemanticException( 
ErrorMsg.ALTER_VIEW_AS_SELECT_NOT_EXIST.getMsg(viewNotExistErrorMsg));
+      throw new 
SemanticException(ErrorMsg.ALTER_VIEW_AS_SELECT_NOT_EXIST.getMsg(viewNotExistErrorMsg));

Review comment:
       Removed.

##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/MapBuilder.java
##########
@@ -17,51 +17,53 @@
  */
 package org.apache.hadoop.hive.ql.metadata.formatting;
 
-import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.Map;
 
 /**
  * Helper class to build Maps consumed by the JSON formatter.  Only
  * add non-null entries to the Map.
  */
-public class MapBuilder {
-    private Map<String, Object> map = new LinkedHashMap<String, Object>();
+public final class MapBuilder {
+  private Map<String, Object> map = new LinkedHashMap<String, Object>();
 
-    private MapBuilder() {}
+  private MapBuilder() {
+  }
 
-    public static MapBuilder create() {
-        return new MapBuilder();
-    }
+  public static MapBuilder create() {
+    return new MapBuilder();
+  }
 
-    public MapBuilder put(String name, Object val) {
-        if (val != null)
-            map.put(name, val);
-        return this;
+  public MapBuilder put(String name, Object val) {
+    if (val != null) {
+      map.put(name, val);
     }
+    return this;
+  }
 
-    public MapBuilder put(String name, boolean val) {
-        map.put(name, Boolean.valueOf(val));
-        return this;
-    }
+  public MapBuilder put(String name, boolean val) {
+    map.put(name, Boolean.valueOf(val));
+    return this;
+  }
 
-    public MapBuilder put(String name, int val) {
-        map.put(name, Integer.valueOf(val));
-        return this;
-    }
+  public MapBuilder put(String name, int val) {
+    map.put(name, Integer.valueOf(val));
+    return this;
+  }
 
-    public MapBuilder put(String name, long val) {
-        map.put(name, Long.valueOf(val));
-        return this;
-    }
+  public MapBuilder put(String name, long val) {
+    map.put(name, Long.valueOf(val));
+    return this;
+  }
 
-    public <T> MapBuilder put(String name, T val, boolean use) {
-        if (use)
-            put(name, val);
-        return this;
+  public <T> MapBuilder put(String name, T val, boolean use) {
+    if (use) {
+      put(name, val);
     }
+    return this;
+  }
 
-    public Map<String, Object> build() {
-        return map;
-    }
+  public Map<String, Object> build() {
+    return map;
+  }

Review comment:
       Removed.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org