pvary commented on PR #4188:
URL: https://github.com/apache/amoro/pull/4188#issuecomment-4278827953

   @lintingbin, @zhangwl9: I don’t think the RowGroup‑merging feature will make 
it into the Iceberg codebase. The core issue is that while this approach merges 
small files and reduces the file count, it does not reduce the number of 
RowGroups—they remain unchanged. As a result, the performance penalty of 
scanning Parquet files with many RowGroups still exists, so this type of 
compaction doesn’t really solve the underlying problem.


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

Reply via email to