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

Kevin Clark commented on THRIFT-265:
------------------------------------

Ok, seeing as everyone agrees this version of the patch is a good thing, but 
might still like to discuss the tunable parameter, could I commit this and if 
you feel strongly another ticket could be opened about the tunable param? I'd 
like to get this in, and it feels like maybe the good is being sacrificed to 
debate the perfect.

> Buffer bloat in TNonblockingServer
> ----------------------------------
>
>                 Key: THRIFT-265
>                 URL: https://issues.apache.org/jira/browse/THRIFT-265
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Library (C++)
>            Reporter: Erik Frey
>            Assignee: Erik Frey
>            Priority: Minor
>         Attachments: buffer_reset.patch, buffer_reset_v2.patch, graph.png
>
>
> TNonBlockingServer never resets the lengths of the buffers it maintains for 
> reading and writing.  Servers with a long life and many concurrent 
> connections eventually generate an overhead that can reach into the 
> gigabytes, particularly in services that have varied message sizes.

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