Re: [PR] Core: Remove synchronization from BitmapPositionDeleteIndex [iceberg]

2023-11-20 Thread via GitHub
nastra commented on PR #9119: URL: https://github.com/apache/iceberg/pull/9119#issuecomment-1820369632 thanks @aokolnychyi for fixing this and @amogh-jahagirdar for reviewing! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Core: Remove synchronization from BitmapPositionDeleteIndex [iceberg]

2023-11-20 Thread via GitHub
nastra merged PR #9119: URL: https://github.com/apache/iceberg/pull/9119 -- 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:

Re: [PR] Core: Remove synchronization from BitmapPositionDeleteIndex [iceberg]

2023-11-20 Thread via GitHub
aokolnychyi closed pull request #9119: Core: Remove synchronization from BitmapPositionDeleteIndex URL: https://github.com/apache/iceberg/pull/9119 -- 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

Re: [PR] Core: Remove synchronization from BitmapPositionDeleteIndex [iceberg]

2023-11-20 Thread via GitHub
aokolnychyi commented on PR #9119: URL: https://github.com/apache/iceberg/pull/9119#issuecomment-1820181900 @nastra @wypoon, could you check whether my analysis was correct? Let me know if I missed something. cc @amogh-jahagirdar @singhpk234 @Fokko as well -- This is an automated

[PR] Core: Remove synchronization from BitmapPositionDeleteIndex [iceberg]

2023-11-20 Thread via GitHub
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,