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 2022/08/01 14:48:27 UTC

[GitHub] [pulsar] shibd opened a new pull request, #16903: [client][c++] Enable client memory limit controller by default.

shibd opened a new pull request, #16903:
URL: https://github.com/apache/pulsar/pull/16903

   ### Motivation
   
   #13306
   
   ### Modifications
   
   - Enable C++ client memory limit controller by default.
   - Git ignore `.cmake` folder
   
   ### Documentation
   
   - [x] `doc-not-needed` 
   (Please explain why)
    
   


-- 
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] shibd commented on pull request #16903: [refactor][c++ client] Enable client memory limit controller by default.

Posted by GitBox <gi...@apache.org>.
shibd commented on PR #16903:
URL: https://github.com/apache/pulsar/pull/16903#issuecomment-1201953301

   > Currently we cannot disable `maxPendingMessages` for C++ client, see the failed tests. In addition, C++ client doesn't implement memory limit for consumer. We need to implement PIP-74 for C++ client before that.
   > 
   > Even for producer, there was an issue for memory limit, see #11910. You can verify this issue first.
   
   I get it, thanks. I will perfect the memory limit first.


-- 
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] shibd closed pull request #16903: [refactor][c++ client] Enable client memory limit controller by default.

Posted by GitBox <gi...@apache.org>.
shibd closed pull request #16903: [refactor][c++ client] Enable client memory limit controller by default.
URL: https://github.com/apache/pulsar/pull/16903


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