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/08/12 20:27:39 UTC

[GitHub] [pulsar] sijia-w opened a new pull request #11653: [docs] [ISSUE 11380] Update code example and description

sijia-w opened a new pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653


   Fixes #11380 
   
   It seems that the changed two file share common content on the usage of message retention policies. Do we need to keep the examples of the commands in both files?


-- 
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] Huanli-Meng commented on a change in pull request #11653: [docs] [ISSUE 11380] Update code example and description

Posted by GitBox <gi...@apache.org>.
Huanli-Meng commented on a change in pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653#discussion_r688398275



##########
File path: site2/docs/admin-api-namespaces.md
##########
@@ -400,8 +392,7 @@ admin.namespaces().unloadNamespaceBundle(namespace, bundle)
 
 #### Split namespace bundles
 
-Each namespace bundle can contain multiple topics and each bundle can be served by only one broker. 
-If a single bundle is creating an excessive load on a broker, an admin splits the bundle using this command permitting one or more of the new bundles to be unloaded thus spreading the load across the brokers.
+One namespace bundle can contain multiple topics but can be served by only one broker. If a single bundle is creating an excessive load on a broker, an admin can split the bundle using the command below, permitting one or more of the new bundles to be unloaded, thus spreading the load across the brokers.

Review comment:
       I think "balancing the load across brokers" is more appropriate.




-- 
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] sijia-w commented on pull request #11653: [docs] [ISSUE 11380] Update code example and description

Posted by GitBox <gi...@apache.org>.
sijia-w commented on pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653#issuecomment-897946101


   @Jennifer88huang Please help review the changes. Thank you.


-- 
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] sijie merged pull request #11653: [docs] [ISSUE 11380] Update code example and description

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


   


-- 
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] sijia-w commented on pull request #11653: [docs] [ISSUE 11380] Update code example and description

Posted by GitBox <gi...@apache.org>.
sijia-w commented on pull request #11653:
URL: https://github.com/apache/pulsar/pull/11653#issuecomment-898733698


   @Huanli-Meng Thanks for your comment. Updated. PTAL.


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