----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21663/ -----------------------------------------------------------
(Updated May 20, 2014, 6:24 a.m.) Review request for kafka. Summary (updated) ----------------- Patch for KAFKA-1456 Bugs: KAFKA-1432 and KAFKA-1456 https://issues.apache.org/jira/browse/KAFKA-1432 https://issues.apache.org/jira/browse/KAFKA-1456 Repository: kafka Description ------- Fix LZ4HC case object typo Revert unnecessary config changes, change server log folder to existing directory Add test cases for LZ4 and LZ4HC message compression Add LZ4 and LZ4HC codec wrappers to client Diffs (updated) ----- build.gradle b3bbd775366e29abb960e11b0b613b3a85ac4747 clients/src/main/java/org/apache/kafka/common/record/CompressionType.java c557e44b68bc201c7c7e8a31e9eb2f74c359762b clients/src/main/java/org/apache/kafka/common/record/Compressor.java 6ae3d06ac6d27e4ecee3f5b4a18e2615d0581442 clients/src/main/java/org/apache/kafka/common/record/Record.java ce1177ef3c851572a3e3268a7a075e7db56fc3f7 config/producer.properties 52a76114f5d0968ca0ff01b3ab1a45450c0a1e33 core/src/main/scala/kafka/message/CompressionCodec.scala 8762a790af2ab7f60a5e4e19955c7eb7ad7bc749 core/src/main/scala/kafka/message/CompressionFactory.scala ca833ee317975548b3e0e91f010af36bce2af09b core/src/main/scala/kafka/message/Message.scala 52c082f5213ba48a39bf0d17e4ced2714276f0e0 core/src/test/scala/unit/kafka/message/MessageCompressionTest.scala ed22931f24fed3530d374b56495a5e4bc89c72f2 system_test/producer_perf/bin/run-compression-test-all.sh PRE-CREATION system_test/producer_perf/config/server.properties 9f8a633ab2f92862e7e7bcc40d699daf00c69e23 Diff: https://reviews.apache.org/r/21663/diff/ Testing ------- Thanks, James Oliver