deniskuzZ commented on code in PR #4520:
URL: https://github.com/apache/hive/pull/4520#discussion_r1302616633
##########
ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java:
##########
@@ -271,6 +300,12 @@ private void generateActualTasks(HiveConf conf,
List<Task<?>> resTsks,
Utilities.FILE_OP_LOGGER.debug("merge resolver will merge " +
status[i].getPath());
work.resolveDynamicPartitionStoredAsSubDirsMerge(conf,
status[i].getPath(), tblDesc,
aliases, pDesc);
+ // Do not add input file since its already added when the manifest
file is present.
+ if (!manifestFilePresent) {
+ List<Path> inputPaths = work.getPaths();
+ inputPaths.add(status[i].getPath());
Review Comment:
should this be done inside the loop? work.setInputPaths keeps changing...
--
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
