Jun,

My apologies for not catching the checkstyle issues before. Should be fine
now. Thanks for the prompt reviews.

On Sun, Apr 19, 2015 at 1:14 AM, Ashish Singh <asi...@cloudera.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31369/
>   Review request for kafka.
> By Ashish Singh.
>
> *Updated April 19, 2015, 8:14 a.m.*
>  *Bugs: * KAFKA-1982 <https://issues.apache.org/jira/browse/KAFKA-1982>
>  *Repository: * kafka
> Description
>
> KAFKA-1982: change kafka.examples.Producer to use the new java producer
>
>   Diffs (updated)
>
>    - 
> clients/src/main/java/org/apache/kafka/common/serialization/IntegerDeserializer.java
>    (PRE-CREATION)
>    - 
> clients/src/main/java/org/apache/kafka/common/serialization/IntegerSerializer.java
>    (PRE-CREATION)
>    - 
> clients/src/test/java/org/apache/kafka/common/serialization/SerializationTest.java
>    (f5cd61c1aa9433524da0b83826a766389de68a0b)
>    - examples/README (53db6969b2e2d49e23ab13283b9146848e37434e)
>    - examples/src/main/java/kafka/examples/Consumer.java
>    (13135b954f3078eeb7394822b0db25470b746f03)
>    - examples/src/main/java/kafka/examples/KafkaConsumerProducerDemo.java
>    (1239394190fe557e025fbd8f3803334402b0aeea)
>    - examples/src/main/java/kafka/examples/Producer.java
>    (96e98933148d07564c1b30ba8e805e2433c2adc8)
>    - examples/src/main/java/kafka/examples/SimpleConsumerDemo.java
>    (0d66fe5f8819194c8624aed4a21105733c20cc8e)
>
> View Diff <https://reviews.apache.org/r/31369/diff/>
>



-- 

Regards,
Ashish

Reply via email to