chinnaraolalam commented on code in PR #5000:
URL: https://github.com/apache/hive/pull/5000#discussion_r1546692298


##########
ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java:
##########
@@ -344,7 +358,7 @@ private void analyzeLoad(ASTNode ast) throws 
SemanticException {
     }
 
     // make sure the arguments make sense
-    List<FileStatus> files = applyConstraintsAndGetFiles(fromURI, 
ts.tableHandle);
+    List<FileStatus> files = applyConstraintsAndGetFiles(fromURI, 
ts.tableHandle, srcs);

Review Comment:
   What is behaviour of below scenarios before and after fix and what exception 
it is throwing
   
   Partitioned - Local path
   Partitioned - HDFS path
   Non-Partitioned - Local path
   Non-Partitioned - HDFS path



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