RussellSpitzer commented on PR #17205: URL: https://github.com/apache/iceberg/pull/17205#issuecomment-4982280593
> Since `RewritePositionDeleteFilesSparkAction` already cleans up dangling deletes during routine compaction—which we should be running regularly anyway to keep the table healthy—I'm not sure the benefits here are worth the added complexity and cost. I think this is a good point here, are we saying that users of V2 should run two different operations? Rewrite Position Deletes and Remove Dangling Deletes? Currently I think we have a pretty straightforward On V2 - RewritePosition On V3 - DropDangling I'm not opposed to adding this functionality but it does seem like we already have an answer for maintenance on position deletes -- 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]
