Mailing-List: contact kafka-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: kafka-dev@incubator.apache.org
Date: Sun, 6 Nov 2011 18:57:52 +0000 (UTC)
From: "Neha Narkhede (Commented) (JIRA)" <jira@apache.org>
To: kafka-dev@incubator.apache.org
Message-ID: 
 <1461904138.5045.1320605872009.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <1621367294.3291.1320516291765.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (KAFKA-187) Add Snappy Compression as a Codec
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/KAFKA-187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13145074#comment-13145074 ] 

Neha Narkhede commented on KAFKA-187:
-------------------------------------

1. I think we might as well do the refactoring it as part of this patch. Copy pasting code doesn't seem like a good idea. 
2. Yes. Same JIRA as this one. We might as well think through all the issues with supporting multiple codecs cleanly now, rather than later.
3. I've updated the perf patch. Do you want to submit a patch for the unit test ?
                
> Add Snappy Compression as a Codec
> ---------------------------------
>
>                 Key: KAFKA-187
>                 URL: https://issues.apache.org/jira/browse/KAFKA-187
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Joe Stein
>         Attachments: kafka-187.patch
>
>
> My thoughts are a new trait CompressionDependencies for KafkaProject.scala, adding snappy as the first library

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira