----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28108/#review62349 -----------------------------------------------------------
We have multiple functions for creating znodes in ZkUtils. persistent, epehmeral, sequence, etc... Does it make sense to put the validation in a function and use it in all "create" functions? - Gwen Shapira On Nov. 16, 2014, 7:49 p.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28108/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2014, 7:49 p.m.) > > > Review request for kafka. > > > Repository: kafka > > > Description > ------- > > KAFKA-1664: Kafka does not properly parse multiple ZK nodes with non-root > chroot > > > Diffs > ----- > > core/src/main/scala/kafka/utils/ZkUtils.scala > 56e3e88e0cc6d917b0ffd1254e173295c1c4aabd > > Diff: https://reviews.apache.org/r/28108/diff/ > > > Testing > ------- > > Tested with and without the fix. > > > Thanks, > > Ashish Singh > >
