iamsanjaymalakar opened a new pull request, #25296:
URL: https://github.com/apache/pulsar/pull/25296

   ### Motivation
   `BlobStoreBackedReadHandleImplV2#closeAsync()` closes indices and 
dataStreams sequentially inside a single try/catch. If closing one resource 
throws, the method exits early and the remaining resources are never closed. 
This can leak file descriptors and leave resources open when offload read 
handles are closed.
   
   ### Modifications
   - Update `closeAsync()` to attempt closing all `OffloadIndexBlockV2` 
instances and all `DataInputStreams`.
   - Collect the first `IOException` and attach subsequent failures via 
suppressed exceptions.
   - Complete the returned `CompletableFuture` exceptionally if any close 
operation fails; otherwise mark the handle Closed and complete normally.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to