[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-28 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r717214760 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -94,56 +104,7

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-27 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r717215049 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedInputStreamImpl.java ## @@ -115,7 +115,

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-27 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r717214760 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -94,56 +104,7

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-24 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715999390 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -94,56 +104,7

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-24 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715998203 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedInputStreamImpl.java ## @@ -115,7 +115,

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-24 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715997737 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedInputStreamImpl.java ## @@ -115,7 +115,

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-24 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715997462 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -94,56 +104,7

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-24 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715997136 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -94,56 +104,7

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-23 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r715309075 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -54,6 +56,15

[GitHub] [pulsar] zymap commented on a change in pull request #12123: Fix the potential race condition in the BlobStore readhandler

2021-09-21 Thread GitBox
zymap commented on a change in pull request #12123: URL: https://github.com/apache/pulsar/pull/12123#discussion_r713600965 ## File path: tiered-storage/jcloud/src/main/java/org/apache/bookkeeper/mledger/offload/jcloud/impl/BlobStoreBackedReadHandleImpl.java ## @@ -105,6 +118,1