Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Kafka >> mail # dev >> Review Request 24214: Patch for KAFKA-1374

Copy link to this message
Re: Review Request 24214: Patch for KAFKA-1374

This is an automatically generated e-mail. To reply, visit:
Thanks for the patch. Some comments below.

    We probably should just record the compressed size in stats.recopyMessage() since that's the size actually got copied over.


    Could we use Compressor.putRecord? Then,we don't have to worry about the details of the message format.


    Would it be better to make this a parameterized test so that we can test all compression codec?
- Jun Rao
On Aug. 3, 2014, 5:40 p.m., Manikumar Reddy O wrote: