fishy commented on PR #2818: URL: https://github.com/apache/thrift/pull/2818#issuecomment-1594888629
> Looks reasonable. But a naiive question: is thrift currently supporting any data size above 4GB? From what I understand, on protocol level, all of [TBinaryProtocol](https://github.com/apache/thrift/blob/master/doc/specs/thrift-binary-protocol.md), [TCompactProtocol](https://github.com/apache/thrift/blob/master/doc/specs/thrift-compact-protocol.md), and [THeaderProtocol](https://github.com/apache/thrift/blob/master/doc/specs/HeaderFormat.md) only support 32-bit lengths. TFramedTransport does not have a spec (as far as I could tell) but the implementations should also all be limiting to 32-bit. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
