deniskuzZ commented on code in PR #5789:
URL: https://github.com/apache/hive/pull/5789#discussion_r2092576941
##########
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java:
##########
@@ -1368,7 +1368,7 @@ public static void
createMRWorkForMergingFiles(FileSinkOperator fsInput,
// NOTE: we should gather stats in MR1 rather than MR2 at merge job since
we don't
// know if merge MR2 will be triggered at execution time
MoveWork dummyMv = null;
- if (srcMmWriteId == null) {
+ if (srcMmWriteId == null && !fsInputDesc.isIcebergTable()) {
Review Comment:
@SourabhBadhya does it look valid to you?
What I noticed is that we create some dummy moveWork that tries to move an
empty `.hive-staging../-tmp.-ext` that doesn't exist.
In case of `UNION ALL` we create UNION subdirs under `.hive-staging../-ext`
that cause the root table folder wipe.
--
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
