----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21240/ -----------------------------------------------------------
(Updated May 14, 2014, 9:40 p.m.) Review request for kafka. Bugs: KAFKA-1179 https://issues.apache.org/jira/browse/KAFKA-1179 Repository: kafka Description (updated) ------- KAFKA-1179. createMessageStreams() in javaapi.ZookeeperConsumerConnector does not throw. Added MessageStreamsExistsException , Changed ZookeeperConsumerConnector to throw new exception and added a unit test part of testBasic. Diffs (updated) ----- core/src/main/scala/kafka/common/MessageStreamsExistsException.scala PRE-CREATION core/src/main/scala/kafka/consumer/ZookeeperConsumerConnector.scala 1dde4fcdd7004af798e9eac8dde289575e99fd11 core/src/main/scala/kafka/javaapi/consumer/ZookeeperConsumerConnector.scala 1f95d9b6a560b353dbd7fe8ad08495d47f9e1ac2 core/src/test/scala/unit/kafka/consumer/ZookeeperConsumerConnectorTest.scala 96fa0bd98141f36e81737865d9d6a4451eedca7c core/src/test/scala/unit/kafka/javaapi/consumer/ZookeeperConsumerConnectorTest.scala 20e8efe5251670259a9ecb8342afd225a02b3422 Diff: https://reviews.apache.org/r/21240/diff/ Testing ------- Thanks, Sriharsha Chintalapani