----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36570/#review92323 -----------------------------------------------------------
Ship it! LGTM. Just a small nit. core/src/main/scala/kafka/admin/AdminUtils.scala (lines 249 - 251) <https://reviews.apache.org/r/36570/#comment146413> Nit: Our code standard includes not using curly brackets on a single line "if" block. core/src/main/scala/kafka/admin/TopicCommand.scala (lines 88 - 90) <https://reviews.apache.org/r/36570/#comment146412> Nit: Our code standard includes not using curly brackets on a single line "if" block. - Gwen Shapira On July 20, 2015, 5:37 p.m., Grant Henke wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36570/ > ----------------------------------------------------------- > > (Updated July 20, 2015, 5:37 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-2337 > https://issues.apache.org/jira/browse/KAFKA-2337 > > > Repository: kafka > > > Description > ------- > > KAFKA-2337: Verify that metric names will not collide when creating new topics > > > Diffs > ----- > > core/src/main/scala/kafka/admin/AdminUtils.scala > f06edf41c732a7b794e496d0048b0ce6f897e72b > core/src/main/scala/kafka/admin/TopicCommand.scala > a90aa8787ff21b963765a547980154363c1c93c6 > core/src/main/scala/kafka/common/Topic.scala > 32595d6fe432141119db26d3b5ebe229aac40805 > core/src/test/scala/unit/kafka/admin/AdminTest.scala > 252ac813c8df1780c2dc5fa9e698fb43bb6d5cf8 > core/src/test/scala/unit/kafka/common/TopicTest.scala > 79532c89c41572ba953c4dc3319a05354927e961 > > Diff: https://reviews.apache.org/r/36570/diff/ > > > Testing > ------- > > > Thanks, > > Grant Henke > >