[GitHub] thrift issue #1375: THRIFT-4346: Allow ZlibTransportFactory to wrap other fa...

2017-09-27 Thread ykhrustalev
Github user ykhrustalev commented on the issue: https://github.com/apache/thrift/pull/1375 thanks @jeking3 , duly noted ---

[GitHub] thrift issue #1375: THRIFT-4346: Allow ZlibTransportFactory to wrap other fa...

2017-09-26 Thread ykhrustalev
Github user ykhrustalev commented on the issue: https://github.com/apache/thrift/pull/1375 It is not quite obvious what are the reasons for CI failures but they don't seem to be related to the patch. ---

[GitHub] thrift pull request #1375: THRIFT-4346: Allow ZlibTransportFactory to wrap o...

2017-09-25 Thread ykhrustalev
GitHub user ykhrustalev opened a pull request: https://github.com/apache/thrift/pull/1375 THRIFT-4346: Allow ZlibTransportFactory to wrap other factories It is a case for when zlib is used with conjunction to other transport, for example framed transport. You can merge