Re: [PR] Core: Fix silent DV conflict when conflictDetectionFilter doesn't cover all touched partitions [iceberg]

["yadavay-amzn (via GitHub)" <gi...@apache.org>]

yadavay-amzn commented on PR #15966:
URL: https://github.com/apache/iceberg/pull/15966#issuecomment-4974193973

   This looks like it was auto-closed by the stale bot rather than on merit - the bot's own note says as much. The fix and the green-CI reproducer here still apply, and the underlying correctness bug still appears to be live on main: when a concurrent row-level writer uses a `conflictDetectionFilter` narrower than the set of partitions the current commit writes DVs into, two DVs can land on the same data file, after which every subsequent scan fails with `Can't index multiple DVs`.
   
   @bvolpato would you like to reopen and revive this? If it's easier, I'm happy to help carry it forward - rebase onto current main and shepherd it through review, with credit to you. Given it can leave a table unreadable, it seems worth landing.
   


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org