----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21899/#review46183 -----------------------------------------------------------
Ship it! Ship It! - Neha Narkhede On June 16, 2014, 9:19 p.m., Sriharsha Chintalapani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21899/ > ----------------------------------------------------------- > > (Updated June 16, 2014, 9:19 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1382 > https://issues.apache.org/jira/browse/KAFKA-1382 > > > Repository: kafka > > > Description > ------- > > KAFKA-1382. Update zkVersion on partition state update failures. Changes as > per Jun's suggestions. > > > Diffs > ----- > > core/src/main/scala/kafka/cluster/Partition.scala > a9c0465b61c7f955e9b0ab93139ca5ef628585c4 > core/src/main/scala/kafka/controller/KafkaController.scala > 8af48ab500779d3d851d25050e1308f5e7b588a6 > core/src/main/scala/kafka/controller/PartitionStateMachine.scala > e29e47003ab000fe5560d6f5aabcc8c38632c852 > core/src/main/scala/kafka/controller/ReplicaStateMachine.scala > 2f0f29d9b76d847700bb64d6d54515b6a926a253 > core/src/main/scala/kafka/utils/ReplicationUtils.scala > eb538377dc1de056703d8d96447f47ff277ecf0a > core/src/main/scala/kafka/utils/ZkUtils.scala > 1a23eb43637662879e447cbda8e84a762cdd6889 > core/src/test/scala/unit/kafka/utils/ReplicationUtilsTest.scala > f3649808e452b56df39904297a4bfe03f1a776bc > > Diff: https://reviews.apache.org/r/21899/diff/ > > > Testing > ------- > > > Thanks, > > Sriharsha Chintalapani > >