[ 
https://issues.apache.org/jira/browse/THRIFT-357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12679088#action_12679088
 ] 

Alexander Shigin commented on THRIFT-357:
-----------------------------------------

Buffer memory limit has type uint32_t, not size_t (like connection stack limit)?
{code}
+  static const size_t CONNECTION_STACK_LIMIT = 1024;
+  static const uint32_t IDLE_BUFFER_MEM_LIMIT = 8192;
{code}

> Buffer and Connection bloat in TNonBlockingServer
> -------------------------------------------------
>
>                 Key: THRIFT-357
>                 URL: https://issues.apache.org/jira/browse/THRIFT-357
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Library (C++)
>            Reporter: Anthony Giardullo
>            Priority: Minor
>         Attachments: buffer_limit.patch
>
>
> This patch helps to reduce the amount of memory allocated by 
> TNonBlockingServer.   It uses a configurable limit on the number of idle 
> connections to keep around, and has a configurable limit on the amount of 
> memory to keep allocated in the readBuffer when the connection is closed.  
> This fix is in use at Facebook and has significantly reduced the amount of 
> memory used by Scribe (https://sourceforge.net/projects/scribeserver/).
> See THRIFT-265 for another memory usage fix which reallocates input/output 
> buffers every 512 reads/writes.  THRIFT-265(and THRIFT-354) and this patch 
> can be used together to tune the amount of memory used.  The buffer_limit 
> patch is especially useful for the case when there are many idle TConnections 
> with large buffers allocated.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to