You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/06/16 03:03:42 UTC

[GitHub] [bookkeeper] horizonzy opened a new pull request, #3337: Fix underReplicatedLedgerTotalSize calculate problem.

horizonzy opened a new pull request, #3337:
URL: https://github.com/apache/bookkeeper/pull/3337

   Descriptions of the changes in this PR:
   ```
     LongAdder underReplicatedSize = new LongAdder();
           FutureUtils.processList(
                   Lists.newArrayList(ledgers),
                   ledgerId ->
                       ledgerManager.readLedgerMetadata(ledgerId).whenComplete((metadata, exception) -> {
                           if (exception == null) {
                               underReplicatedSize.add(metadata.getValue().getLength());
                           }
                       }), null);
           underReplicatedLedgerTotalSize.registerSuccessfulValue(underReplicatedSize.longValue());
   ```
   `FutureUtils.processList`  is async process, should record `underReplicatedLedgerTotalSize` when it completed.


-- 
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-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] hangc0276 merged pull request #3337: Fix underReplicatedLedgerTotalSize calculate problem.

Posted by GitBox <gi...@apache.org>.
hangc0276 merged PR #3337:
URL: https://github.com/apache/bookkeeper/pull/3337


-- 
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-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] StevenLuMT commented on pull request #3337: Fix underReplicatedLedgerTotalSize calculate problem.

Posted by GitBox <gi...@apache.org>.
StevenLuMT commented on PR #3337:
URL: https://github.com/apache/bookkeeper/pull/3337#issuecomment-1194895403

   rerun failure checks


-- 
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-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] StevenLuMT commented on pull request #3337: Fix underReplicatedLedgerTotalSize calculate problem.

Posted by GitBox <gi...@apache.org>.
StevenLuMT commented on PR #3337:
URL: https://github.com/apache/bookkeeper/pull/3337#issuecomment-1194894469

   rerun failure checks


-- 
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-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] horizonzy closed pull request #3337: Fix underReplicatedLedgerTotalSize calculate problem.

Posted by GitBox <gi...@apache.org>.
horizonzy closed pull request #3337: Fix underReplicatedLedgerTotalSize calculate problem.
URL: https://github.com/apache/bookkeeper/pull/3337


-- 
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-unsubscribe@bookkeeper.apache.org

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