----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19953/#review39550 -----------------------------------------------------------
clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java <https://reviews.apache.org/r/19953/#comment71956> This logic is fairly involved. Let's add unit tests for each of the conditions mentioned here clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java <https://reviews.apache.org/r/19953/#comment71954> Could we just call this cluster instead of curCluster? clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java <https://reviews.apache.org/r/19953/#comment71959> random->any - Neha Narkhede On April 2, 2014, 6:49 p.m., Jun Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19953/ > ----------------------------------------------------------- > > (Updated April 2, 2014, 6:49 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1303 > https://issues.apache.org/jira/browse/KAFKA-1303 > > > Repository: kafka > > > Description > ------- > > Changes are described in the comment of selectMetadataDestination(). > > > 1. Added the logic to use bootstrap cluster if no node can be selected from > the real cluster. 2. Fixed the issue that node is not selected randomally. > > > Diffs > ----- > > > clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java > 855ae84f14aa91653b3fa855c2af40560323f42a > > Diff: https://reviews.apache.org/r/19953/diff/ > > > Testing > ------- > > > Thanks, > > Jun Rao > >
