----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27634/#review60188 -----------------------------------------------------------
Awesome. Its building successfully now. core/src/main/scala/kafka/utils/Utils.scala <https://reviews.apache.org/r/27634/#comment101524> I think I'm not seeing why we need this. Shouldn't Scala's JavaConversion class handle this exact case? core/src/test/scala/kafka/log/LogConfigTest.scala <https://reviews.apache.org/r/27634/#comment101525> Can you add a test that shows that we indeed fail validation when creating LogConfig with invalid values? core/src/test/scala/unit/kafka/integration/UncleanLeaderElectionTest.scala <https://reviews.apache.org/r/27634/#comment101532> String.valueOf(false) should evaluate to "false", right? Why do we need "nottrue"? If String.valueOf(false) no longer works as expected, it looks like a bug waiting to happen... core/src/test/scala/unit/kafka/integration/UncleanLeaderElectionTest.scala <https://reviews.apache.org/r/27634/#comment101527> why are we removing a test? - Gwen Shapira On Nov. 6, 2014, 4:12 p.m., Dmytro Kostiuchenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27634/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2014, 4:12 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1667 > https://issues.apache.org/jira/browse/KAFKA-1667 > > > Repository: kafka > > > Description > ------- > > KAFKA-1667 Fixed bugs in LogConfig. Added test and documentation > > > KAFKA-1667 Updated tests to reflect new boolean property parsing logic > > > KAFKA-1667 renamed methods to match naming convention > > > Diffs > ----- > > clients/src/main/java/org/apache/kafka/common/config/ConfigDef.java > c4cea2cc072f4db4ce014b63d226431d3766bef1 > core/src/main/scala/kafka/admin/TopicCommand.scala > 0b2735e7fc42ef9894bef1997b1f06a8ebee5439 > core/src/main/scala/kafka/log/LogConfig.scala > e48922a97727dd0b98f3ae630ebb0af3bef2373d > core/src/main/scala/kafka/utils/Utils.scala > 23aefb4715b177feae1d2f83e8b910653ea10c5f > core/src/test/scala/kafka/log/LogConfigTest.scala PRE-CREATION > core/src/test/scala/unit/kafka/integration/UncleanLeaderElectionTest.scala > f44568cb25edf25db857415119018fd4c9922f61 > > Diff: https://reviews.apache.org/r/27634/diff/ > > > Testing > ------- > > > Thanks, > > Dmytro Kostiuchenko > >
