wwbmmm opened a new pull request, #3035:
URL: https://github.com/apache/brpc/pull/3035

   ### What problem does this PR solve?
   
   Issue Number: null
   
   Problem Summary: redis request or response may have a too large size that 
cause the server to consume too much memory or even crash.
   
   ### What is changed and the side effects?
   
   Changed: Add a limit for allocation size
   
   Side effects:
   - Performance effects: no
   
   - Breaking backward compatibility: Some redis request/response may fail to 
parse if the data size exceed the default value of redis_max_allocation_size 
(64M). User can adjust the flag redis_max_allocation_size for their senarios.
   
   ---
   ### Check List:
   - Please make sure your changes are compilable.
   - When providing us with a new feature, it is best to add related tests.
   - Please follow [Contributor Covenant Code of 
Conduct](https://github.com/apache/brpc/blob/master/CODE_OF_CONDUCT.md).
   


-- 
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: dev-unsubscr...@brpc.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org

Reply via email to