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

Nate McCall commented on THRIFT-820:
------------------------------------

FWIW, we switched to creating new TProtocolBuffer objects after each 
TProcessor.process call (CASSANDRA-475) so we are not so hot-to-trot on this 
for now. 

If the consensus is to reset the length, that's fine, but then a resetLength() 
method should exposed for such.

> The readLength attribute of TBinaryProtocol is used as an instance variable 
> and is decremented on each call of checkReadLength
> ------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-820
>                 URL: https://issues.apache.org/jira/browse/THRIFT-820
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (Java)
>    Affects Versions: 0.2, 0.3
>            Reporter: Nate McCall
>         Attachments: tbinaryprotocol-length-check-patch-THRIFT-820.txt
>
>
> When coding towards readLength, I saw this more as a check on the message 
> length as it comes in. Perhaps its me, but I just dont see why this should 
> decrement giving the length header for each message.

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