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 2020/10/18 10:02:17 UTC

[GitHub] [pulsar] hangc0276 opened a new pull request #8289: Support limit max subscriptions per topic

hangc0276 opened a new pull request #8289:
URL: https://github.com/apache/pulsar/pull/8289


   Fix #8226 
   
   ### Motivation
   Support limits the max subscriptions per topic of the Pulsar cluster. When the subscriptions reach the max subscriptions of the topic, the broker should reject the new subscription request and reject the consumer to subscribe to this topic.
   
   ### Modifications
   1. Add maxSubscriptionsPerTopic=0 in the broker.conf and don't limit by default.
   2. add limit max subscriptions per topic
   
   ### Verifying this change
   AdminApiTest2#testMaxSubscriptionsPerTopic


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

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

[GitHub] [pulsar] Jennifer88huang merged pull request #8289: Support limit max subscriptions per topic

Posted by GitBox <gi...@apache.org>.
Jennifer88huang merged pull request #8289:
URL: https://github.com/apache/pulsar/pull/8289


   


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

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

[GitHub] [pulsar] Anonymitaet commented on pull request #8289: Support limit max subscriptions per topic

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


   Hi @momo-jun can you follow up on the docs? 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

[GitHub] [pulsar] hangc0276 commented on pull request #8289: Support limit max subscriptions per topic

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


   /pulsarbot run-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.

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

[GitHub] [pulsar] hangc0276 commented on pull request #8289: Support limit max subscriptions per topic

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


   /pulsarbot run-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.

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