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


##########
ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java:
##########
@@ -348,6 +348,17 @@ public TaskInformation(Task task, String path) {
 
   @Override
   public int execute() {
+    if (work != null && work.getLoadTableWork() != null && 
work.getLoadTableWork().isUseAppendForLoad()) {

Review Comment:
   changed & added test for analyse



##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveStorageHandler.java:
##########
@@ -304,6 +305,22 @@ default boolean 
canComputeQueryUsingStats(org.apache.hadoop.hive.ql.metadata.Tab
   default StorageFormatDescriptor getStorageFormatDescriptor(Table table) 
throws SemanticException {
     return null;
   }
+
+  /**
+   * Checks whether the table supports appending data files to the table.
+   * @param table the table
+   * @return true if the table can append files directly to the table
+   * @throws SemanticException in case of any error.
+   */
+  default boolean supportsAppendData(Table table) throws SemanticException {
+    throw new SemanticException(ErrorMsg.LOAD_INTO_NON_NATIVE.getMsg());

Review Comment:
   done, thnx



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to