> On Feb. 5, 2014, 6:57 p.m., Chris Riccomini wrote: > > samza-kafka/src/main/scala/org/apache/samza/system/kafka/KafkaSystemConsumer.scala, > > line 121 > > <https://reviews.apache.org/r/17741/diff/1/?file=470036#file470036line121> > > > > You don't need the {} here, right?
nope, removed and diff updated. Thanks Chris - Steven ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17741/#review33704 ----------------------------------------------------------- On Feb. 5, 2014, 12:13 p.m., Steven Yates wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17741/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2014, 12:13 p.m.) > > > Review request for samza. > > > Bugs: SAMZA-116 > https://issues.apache.org/jira/browse/SAMZA-116 > > > Repository: samza > > > Description > ------- > > A quick patch for Samza-116 > > > Diffs > ----- > > samza-kafka/src/main/scala/org/apache/samza/system/kafka/BrokerProxy.scala > 89730db > > samza-kafka/src/main/scala/org/apache/samza/system/kafka/KafkaSystemConsumer.scala > 56ec964 > > samza-kafka/src/test/scala/org/apache/samza/system/kafka/TestBrokerProxy.scala > bcd8998 > > Diff: https://reviews.apache.org/r/17741/diff/ > > > Testing > ------- > > > Thanks, > > Steven Yates > >
