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 2021/06/26 07:31:21 UTC
[GitHub] [pulsar] 315157973 opened a new pull request #11110: Fix GetListInBundle return all Topics in bundle
315157973 opened a new pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110
### Motivation
https://github.com/apache/pulsar/blob/7824325efc822307f60b38e9d24bdb189e415a06/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/NonPersistentTopics.java#L310-L313
1)
`getListFromBundle` in NonPersistentTopics should only return the list of non-persistent topics under a namespace bundle.
But now it returns all the topics.
2)
Now it will traverse all topics on Broker. When there are a lot of topics, timeouts often occur.
### Modifications
1) Filter out persistent topics
2) Only traverse the topics under the corresponding bundle
### Verifying this change
--
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] codelipenghui merged pull request #11110: Fix GetListInBundle return all Topics in bundle
Posted by GitBox <gi...@apache.org>.
codelipenghui merged pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110
--
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] 315157973 commented on pull request #11110: Fix GetListInBundle return all Topics in bundle
Posted by GitBox <gi...@apache.org>.
315157973 commented on pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110#issuecomment-869743386
> @315157973 does this affect docs?
Does not affect the docs
--
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] codelipenghui merged pull request #11110: Fix GetListInBundle return all Topics in bundle
Posted by GitBox <gi...@apache.org>.
codelipenghui merged pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110
--
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] Anonymitaet commented on pull request #11110: Fix GetListInBundle return all Topics in bundle
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110#issuecomment-869334873
@315157973 does this affect docs?
--
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] 315157973 commented on pull request #11110: Fix GetListInBundle return all Topics in bundle
Posted by GitBox <gi...@apache.org>.
315157973 commented on pull request #11110:
URL: https://github.com/apache/pulsar/pull/11110#issuecomment-869743386
> @315157973 does this affect docs?
Does not affect the docs
--
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