No probs, I just didn't want to keep going backwards and forwards missing the mark with this that's all.
Cheers -------- Original message -------- From: Jakob Homan <[email protected]> Date: To: [email protected],Steve Yates <[email protected]> Subject: Re: Review Request 17875: SAMZA-30 ExponentialSleepThreadStrategy::getNextDelay for class and method seems like it would work? Just wanted to avoid the term 'delivery' since the class could be used for sleeping during other types of operations as well. Does that work for you? On Thu, Feb 20, 2014 at 6:12 PM, Steve Yates <[email protected]>wrote: > Hi Jakob, can I kindly ask you to suggest a fitting name for this utility > class and as to make it descriptive enough yet lends it self to reuse as > discussed prior. > > Many thanks > Steven Yates > > -------- Original message -------- > From: Jakob Homan <[email protected]> > Date: > To: samza <[email protected]>,Jakob Homan > <[email protected]>,Steven > Yates <[email protected]> > Subject: Re: Review Request 17875: SAMZA-30 > > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17875/ > > Looks good except for my being unclear on the class name versus method > name, mentioned below. Sorry for the delay. > > - Jakob Homan > > > On February 15th, 2014, 4:06 a.m. PST, Steven Yates wrote: > > Review request for samza. > By Steven Yates. > Updated Feb. 15, 2014, 4:06 a.m. > > Repository: samza > Description > > SAMZA-30 - Back off in BrokerProxy when fetch fails > Testing > > Some > Diffs > > samza-kafka/src/main/scala/org/apache/samza/system/kafka/BrokerProxy.scala > (34727e9) > samza-kafka/src/main/scala/org/apache/samza/util/ThreadUtil.scala > (PRE-CREATION) > samza-kafka/src/test/scala/org/apache/samza/util/TestThreadUtil.scala > (PRE-CREATION) > View Diff > > File Attachments > > SAMZA-30-4.diff > SAMZA-30-3.diff >
