szehon-ho commented on PR #6581: URL: https://github.com/apache/iceberg/pull/6581#issuecomment-1412448278
Thanks all for reviews. After thinking, this will be good to have, but as the complete 100% position delete removal will be done via minor compaction of delete files (first part of the effort is here: https://github.com/apache/iceberg/pull/6365, more to come). Will work on that first. @amogh-jahagirdar yea I initially thought it would be useful as a standalone action, but some chats with @aokolnychyi and maybe its a bit too tricky for user to know when to run it. Still open though. @manuzhang do you mean integrating this commit with existing partial commits? Initially envisoning a separate commit altogether at the end of RewriteDataFiles. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
