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

Antoine Pitrou commented on THRIFT-5237:
----------------------------------------

I'm updating this now that I've read through the relevant TTransport 
implementations.

In Parquet C++ we're usingĀ {{TMemoryBuffer}} with a known-size buffer that 
we've read ourselves from disk. This means there isn't a way for TTransport 
reads to go past the buffer size, and the max message size limit is not 
relevant to us. Is my understanding correct?

> Implement MAX_MESSAGE_SIZE and consolidate limits into a TConfiguration class
> -----------------------------------------------------------------------------
>
>                 Key: THRIFT-5237
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5237
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C glib - Library, C++ - Library, Java - Library
>            Reporter: Zezeng Wang
>            Assignee: Zezeng Wang
>            Priority: Major
>             Fix For: 0.14.0
>
>          Time Spent: 7.5h
>  Remaining Estimate: 0h
>
> This ticket has two related goals:
> a) to implement a new limit for the maximum message size similar to max 
> frames size etc
> b) consolidate and centralize all limits we have (max msg size,. max frame 
> size and max recursion depth) into one place in the code



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to