You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "crossoverJie (via GitHub)" <gi...@apache.org> on 2023/02/02 14:28:12 UTC
[GitHub] [pulsar] crossoverJie opened a new issue, #19403: Add a new admin API getAllLoadReport()
crossoverJie opened a new issue, #19403:
URL: https://github.com/apache/pulsar/issues/19403
### Search before asking
- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
### Motivation
I want to get the load information of all brokers and currently, I can only do it by a loop call `org.apache.pulsar.client.admin.BrokerStats#getLoadReport`
Also, I need to save additional IP information for all brokers.
### Solution
Add a new admin API `getAllLoadReport()` to get information about all brokers.
### Alternatives
At the moment I can think of two implementation options:
- Internally call the `getLoadReport` API of all brokers to aggregate their data.
- Refer to the `org.apache.pulsar.testclient.BrokerMonitor` implementation to get information from `ZooKeeper`.
https://github.com/apache/pulsar/blob/002fa49692ff4b8b038bf2edeeb3b32ed218e64f/pulsar-testclient/src/main/java/org/apache/pulsar/testclient/BrokerMonitor.java#L51-L55
Any better suggestions?
### Anything else?
_No response_
### Are you willing to submit a PR?
- [X] I'm willing to submit a PR!
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] github-actions[bot] commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1685498223
The issue had no activity for 30 days, mark with Stale label.
--
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] github-actions[bot] commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1635164423
The issue had no activity for 30 days, mark with Stale label.
--
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] Technoboy- commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.
Technoboy- commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1643838992
> @Technoboy- @codelipenghui do we agree on this direction? Or it's just won't do.
>
> I can imagine that "loop call org.apache.pulsar.client.admin.BrokerStats#getLoadReport" doesn't sound too bad.
Ah, sorry for the late response. It's ok to add this kind of the API.
> Refer to the org.apache.pulsar.testclient.BrokerMonitor implementation to get information from ZooKeeper.
Looks better
--
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] tisonkun commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1588554785
@Technoboy- @codelipenghui do we agree on this direction? Or it's just won't do.
I can imagine that "loop call org.apache.pulsar.client.admin.BrokerStats#getLoadReport" doesn't sound too bad.
--
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] github-actions[bot] commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1454960730
The issue had no activity for 30 days, mark with Stale label.
--
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] crossoverJie commented on issue #19403: Add a new admin API getAllLoadReport()
Posted by "crossoverJie (via GitHub)" <gi...@apache.org>.
crossoverJie commented on issue #19403:
URL: https://github.com/apache/pulsar/issues/19403#issuecomment-1644067462
@Technoboy- Please assign it to me, thanks.
--
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