-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20050/
-----------------------------------------------------------
(Updated April 10, 2014, 5:11 p.m.)
Review request for kafka.
Summary (updated)
-----------------
Fix KAFKA-1359: move producer metrics into a centalized class; fix some bugs in
SenderTest
Bugs: KAFKA-1359
https://issues.apache.org/jira/browse/KAFKA-1359
Repository: kafka
Description (updated)
-------
remove un-used imports
move registration to centralized ProducerMetrics; fixed a small bug in
SenderTest
KAFKA-1359.v2
KAFKA-1359.v1
Diffs (updated)
-----
clients/src/main/java/org/apache/kafka/clients/producer/KafkaProducer.java
a6423f4b37a57f0290e2048b764de1218470f4f7
clients/src/main/java/org/apache/kafka/clients/producer/internals/RecordAccumulator.java
ffd13ff00ba0d0d969e2ed130e49053f02481b50
clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java
855ae84f14aa91653b3fa855c2af40560323f42a
clients/src/main/java/org/apache/kafka/common/ProducerMetrics.java
PRE-CREATION
clients/src/main/java/org/apache/kafka/common/network/Selector.java
558f8b47638b354f9c1a30be5d45dc7b61131bea
clients/src/test/java/org/apache/kafka/clients/producer/SenderTest.java
a2b77226f8c58caf632a0f4665bd4e4cd93e643d
clients/src/test/java/org/apache/kafka/common/network/SelectorTest.java
5c5e3d40819e41cab7b52a0eeaee5f2e7317b7b3
Diff: https://reviews.apache.org/r/20050/diff/
Testing
-------
Thanks,
Guozhang Wang