deniskuzZ commented on code in PR #4392:
URL: https://github.com/apache/hive/pull/4392#discussion_r1226680347


##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -795,6 +795,9 @@ public static enum ConfVars {
         "String used as a file extension for output files. \n" +
         "If not set, defaults to the codec extension for text files (e.g. 
\".gz\"), or no extension otherwise."),
 
+    HIVE_TEZ_FOR_LOAD_NATIVE_TABLE("hive.tez.for.load.native.table", false, 
"Whether to use a Tez job always for load" +
+        " queries to native table(like iceberg) even if they support appending 
data files directly"),

Review Comment:
   we should rename it as well
   ````
     /**
      * Extract the native properties of the table which aren't stored in the 
HMS
      * @param table the table
      * @return map with native table level properties
      */
     default Map<String, String> 
getNativeProperties(org.apache.hadoop.hive.ql.metadata.Table table) {
       return new HashMap<>();
     }
   ````



-- 
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.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

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

Reply via email to