Re: [DISCUSSION] Nested compression in Kafka?

2014-10-30 Thread Jun Rao
Yes, I agree that we don't need to support more than 1 level of nesting for the compressed messages. Thanks, Jun On Mon, Oct 27, 2014 at 4:27 PM, Guozhang Wang wangg...@gmail.com wrote: Hello folks, I came across this testComplexCompressDecompress in kafka.message.MessageCompressionTest

Re: [DISCUSSION] Nested compression in Kafka?

2014-10-30 Thread Neha Narkhede
+1 on not having more than one level of compression. On Thu, Oct 30, 2014 at 1:30 PM, Jun Rao jun...@gmail.com wrote: Yes, I agree that we don't need to support more than 1 level of nesting for the compressed messages. Thanks, Jun On Mon, Oct 27, 2014 at 4:27 PM, Guozhang Wang

Re: [DISCUSSION] Nested compression in Kafka?

2014-10-30 Thread Guozhang Wang
Thanks Jun and Neha. I will go ahead and file the JIRA to remove this test case then.

[DISCUSSION] Nested compression in Kafka?

2014-10-27 Thread Guozhang Wang
Hello folks, I came across this testComplexCompressDecompress in kafka.message.MessageCompressionTest while I'm working some consumer decompression optimization. This test checks if nested compression is supported. I remember vaguely that some time ago we decide not to support nested compression