You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/31 04:53:58 UTC

[GitHub] [pulsar] tjiuming opened a new pull request #14958: [enh][broker] add `backlog messages` metrics based on `BrokerEntryMetadata`

tjiuming opened a new pull request #14958:
URL: https://github.com/apache/pulsar/pull/14958


   ### Motivation
   
   Expose precise `backlog messages count` 
   
   ### Modifications
   
   1. add `getBacklogMessages` method in `ManagedCursor`
   2. add `getPublishedMessages` method in `ManagedLedger`
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API: (no)
     - The schema: (no)
     - The default values of configurations: (no)
     - The wire protocol: (no)
     - The rest endpoints: (no)
     - The admin cli options: (no)
     - Anything that affects deployment: (no)
   
   ### Documentation
   
   Check the box below or label this PR directly (if you have committer privilege).
   
   Need to update docs? 
   
   - [x] `doc-required` 
     
     (If you need help on updating docs, create a doc issue)
     
   - [ ] `no-need-doc` 
     
     (Please explain why)
     
   - [ ] `doc` 
     
     (If this PR contains doc changes)
   
   
   


-- 
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] tjiuming edited a comment on pull request #14958: [enh][broker] add `backlog messages` metrics based on `BrokerEntryMetadata`

Posted by GitBox <gi...@apache.org>.
tjiuming edited a comment on pull request #14958:
URL: https://github.com/apache/pulsar/pull/14958#issuecomment-1084074191


   This pr is not finished yet, please take a review and give your advice.


-- 
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] tjiuming commented on pull request #14958: [enh][broker] add `backlog messages` metrics based on `BrokerEntryMetadata`

Posted by GitBox <gi...@apache.org>.
tjiuming commented on pull request #14958:
URL: https://github.com/apache/pulsar/pull/14958#issuecomment-1084074191


   This pr not finished yet, please take a review and give your advice.


-- 
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