----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37357/#review94979 -----------------------------------------------------------
Changes look good. It seems like there quite a few changes in the upstream library, it would probably be good to do more testing than just the unit tests. - Ismael Juma On Aug. 11, 2015, 6:56 p.m., Rajini Sivaram wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37357/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2015, 6:56 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-2421 > https://issues.apache.org/jira/browse/KAFKA-2421 > > > Repository: kafka > > > Description > ------- > > Patch for KAFKA-2421: Upgrade to LZ4 version 1.3 and update reference to > Utils method that was moved to SafeUtils > > > Diffs > ----- > > build.gradle 1b67e628c2fca897177c12b6afad9a8700fffd1f > > clients/src/main/java/org/apache/kafka/common/record/KafkaLZ4BlockInputStream.java > f480da2ae0992855cc860e1ce5cbd11ecfca7bee > > clients/src/main/java/org/apache/kafka/common/record/KafkaLZ4BlockOutputStream.java > 6a2231f4775771932c36df362c88aead3189b7b8 > > Diff: https://reviews.apache.org/r/37357/diff/ > > > Testing > ------- > > > Thanks, > > Rajini Sivaram > >
