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/12/29 10:07:07 UTC

[GitHub] [pulsar] Shoothzj opened a new pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

Shoothzj opened a new pull request #13557:
URL: https://github.com/apache/pulsar/pull/13557


   Fixes #13439 
   ### Motivation
   Now we only constraint `limit` or `limitSize` be set on pulsar-admin, we don't limit it on rest endpoint. So there are situtations that both `limit` and `limitSize` be null
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
     - *Added test method in `BacklogQuotaCompatibilityTest`
   
   ### Documentation
   
   Check the box below or label this PR directly (if you have committer privilege).
   
   Need to update docs? 
     
   - [x] `no-need-doc` 
     
     bug fix, no need doc
   
   
   


-- 
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] Shoothzj commented on pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

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


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] Shoothzj commented on pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

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


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

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 #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

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


   


-- 
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] Shoothzj removed a comment on pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

Posted by GitBox <gi...@apache.org>.
Shoothzj removed a comment on pull request #13557:
URL: https://github.com/apache/pulsar/pull/13557#issuecomment-1002814926


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] Shoothzj commented on pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

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


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] Shoothzj removed a comment on pull request #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

Posted by GitBox <gi...@apache.org>.
Shoothzj removed a comment on pull request #13557:
URL: https://github.com/apache/pulsar/pull/13557#issuecomment-1002834874


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

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 #13557: [issue #13439][backlogQuota] Allow both limit and limitsize be null

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


   


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