aokolnychyi opened a new pull request, #9119: URL: https://github.com/apache/iceberg/pull/9119
This PR removes synchronization from `BitmapPositionDeleteIndex` added in PR #8805. We switched to `ParallelIterable` to parallelize reading of deletes in the original change. However, marking the `delete` method in `BitmapPositionDeleteIndex` synchronized was redundant because `ParallelIterable` outputs elements synchronously. That class internally uses a thread-safe queue to process the input iterables concurrently but does not require extra synchronization in consumers. In other words, there will be only on thread consuming from the iterable that will be populated by multiple threads internally. -- 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
