[GitHub] [pulsar] lhotari opened a new pull request, #15918: [ML] Fix race condition in getManagedLedgerInternalStats when includeLedgerMetadata=true

[GitBox <gi...@apache.org>]

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

   Fixes #15916 
   
   ### Motivation
   
   - `getLedgerMetadata` is an asynchronous operation and the final result shouldn't complete before the
     metadata for all ledgers has been retrieved
   
   ### Modifications
   
   - add CompletableFutures for each retrieval of ledger metadata to a map and wait for completion before completing the result of `getManagedLedgerInternalStats`. 
   - organize operations in a way that all synchronous calls are made before waiting for the asynchronous operations to complete
   
   ### Additional context
   
   managedLedgerInternalStats feature was introduced in #10653 , more additions in #10725


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] lhotari merged pull request #15918: [ML] Fix race condition in getManagedLedgerInternalStats when includeLedgerMetadata=true

[GitBox <gi...@apache.org>]

lhotari merged PR #15918:
URL: https://github.com/apache/pulsar/pull/15918


-- 
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: commits-unsubscribe@pulsar.apache.org

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