deniskuzZ commented on code in PR #5789:
URL: https://github.com/apache/hive/pull/5789#discussion_r2093324416
##########
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:
> Was there any mergeWork created here?
yes
> Ideally this is not the place for table format specific code
MmWriteId is ACID specific :)
--
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]
