Re: Review Request 33199: Coordinator stream

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33199/#review81242 --- Ship it! Went through the latest patch w/ Naveen. LGTM - Yi Pan (D

Re: Review Request 33453: SAMZA-557 Reuse local state in SamzaContainer on clean shutdown

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33453/ --- (Updated April 22, 2015, 9:54 p.m.) Review request for samza, Yan Fang, Chris R

Re: Review Request 33453: SAMZA-557 Reuse local state in SamzaContainer on clean shutdown

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33453/ --- (Updated April 22, 2015, 9:47 p.m.) Review request for samza, Yan Fang, Chris R

Review Request 33453: SAMZA-557 initial working draft - needs refactoring

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33453/ --- Review request for samza, Yan Fang, Chris Riccomini, Naveen Somasundaram, and Yi

Re: Review Request 33305: [SAMZA-655] Add EnvironmentConfigRewriter to samza-api.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33305/#review81210 --- Ship it! Ship It! - Jakob Homan On April 22, 2015, 8:11 a.m., Da

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

> On April 22, 2015, 5:49 p.m., Naveen Somasundaram wrote: > > samza-elasticsearch/src/main/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemProducer.java, > > line 116 > > > > > > If the previous send ha

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33297/#review81197 --- samza-elasticsearch/src/main/java/org/apache/samza/system/elasticse

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33297/#review81184 --- samza-elasticsearch/src/main/java/org/apache/samza/system/elasticse

Re: Request: Speakers for May 5 meet up at LinkedIn in Mountain View, CA

Hi all - I wanted to let you know that the agendahttp://www.meetup.com/Bay-Area-Samza-Meetup/events/221738067/> for this meetup is set. Looks like fun! Hope you can all make it! Thanks, Ed Yakabosky Agenda: 6-6:30PM: Arrival & Networking 6:30-7:15PM: "Clojure with Samza: application arch

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33297/ --- (Updated April 22, 2015, 5:12 p.m.) Review request for samza. Repository: sam

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33297/ --- (Updated April 22, 2015, 5:09 p.m.) Review request for samza. Repository: sam

Re: Review Request 33297: [SAMZA-654] Added ElasticsearchSystemProducer and Factory to output messages into Elasticseach indexes.

> On April 17, 2015, 9:49 p.m., Jakob Homan wrote: > > settings.gradle, line 33 > > > > > > This list of non-Scala projects to check is getting too long and > > difficult to read. Can we just the relevant projects int

Re: Review Request 33305: [SAMZA-655] Add EnvironmentConfigRewriter to samza-api.

> On April 20, 2015, 9:55 p.m., Navina Ramesh wrote: > > Can you update the Samza [configuration > > docs](http://samza.apache.org/learn/documentation/latest/jobs/configuration-table.html) > > with info about this re-writer ? Sure, I've added to the configuration-table for this. - Dan

Re: Review Request 33305: [SAMZA-655] Add EnvironmentConfigRewriter to samza-api.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33305/ --- (Updated April 22, 2015, 3:11 p.m.) Review request for samza. Repository: sam

Re: Review Request 33305: [SAMZA-655] Add EnvironmentConfigRewriter to samza-api.

> On April 17, 2015, 9:23 p.m., Jakob Homan wrote: > > samza-api/src/main/java/org/apache/samza/config/EnvironmentConfigRewriter.java, > > line 41 > > > > > > It may be worth being chatty here in terms of the new confi

Re: Review Request 33305: [SAMZA-655] Add EnvironmentConfigRewriter to samza-api.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33305/ --- (Updated April 22, 2015, 2:51 p.m.) Review request for samza. Repository: sam