aokolnychyi commented on PR #10200: URL: https://github.com/apache/iceberg/pull/10200#issuecomment-2078403034
> Also there is a behavioural change that the previous write rolled deletes @pvary, I don't think that threshold makes sense in this scenario. It was needed because we kept rows and positions in a set in the old writers. We are migrating to `SortingPositionOnlyDeleteWriter` that keeps track of positions in bitmaps. Even millions of position deletes would take KBs on disk. -- 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
