I don't believe that the cross test framework will test zlib changes; not sure if the unit tests do a good job at that either. If anyone else has an opinion on this, let me know. Looks fine to me otherwise, but I don't see any explicit test that would use a zlib transport and flush empty to force this. Is there a way to modify the zlib unit test to force this condition during the test?
[ Full content available at: https://github.com/apache/thrift/pull/1591 ] This message was relayed via gitbox.apache.org for [email protected]
