Re: Review Request 41620: SAMZA-847: update hello-samza master to use Samza 0.10.0

2015-12-21 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41620/ --- (Updated Dec. 22, 2015, 1:04 a.m.) Review request for samza, Yan Fang, Chinmay

Re: Review Request 41620: SAMZA-363; create a remote development branch for hello-samza

2015-12-21 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41620/#review111564 --- Ship it! lgtm! - Navina Ramesh On Dec. 22, 2015, 12:38 a.m., Yi

Review Request 41620: SAMZA-363; create a remote development branch for hello-samza

2015-12-21 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41620/ --- Review request for samza, Yan Fang, Chinmay Soman, Chris Riccomini, Jakob Homan,

Re: CoordinatorStream errors

2015-12-21 Thread Navina Ramesh
Ok. Sounds good. Thanks! On Mon, Dec 21, 2015 at 11:38 AM, Rick Mangi wrote: > HI Navina, > > It stopped happening once I deleted an old checkpoint topic. I think in > the rapid development cycle my checkpoints became invalid. If it happens > again I will save the logs. > > Thanks! > > > > On De

Re: CoordinatorStream errors

2015-12-21 Thread Rick Mangi
HI Navina, It stopped happening once I deleted an old checkpoint topic. I think in the rapid development cycle my checkpoints became invalid. If it happens again I will save the logs. Thanks! > On Dec 21, 2015, at 2:14 PM, Navina Ramesh > wrote: > > Hi Rick, > Can you share the entire log

Re: CoordinatorStream errors

2015-12-21 Thread Navina Ramesh
Hi Rick, Can you share the entire log for this issue? I suspect the concurrent access happens on the bootstrappedSet (LinkedHashSet -> not thread safe) between the Job Coordinator and SamzaAppMaster. When a container fails, the AM tried to read the locality information. If some other container req

Re: Review Request 41580: SAMZA-831: Update Samza 0.10 online documentation

2015-12-21 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41580/#review111516 --- Ship it! Ship It! - Navina Ramesh On Dec. 19, 2015, 1:26 a.m.,

Re: Review Request 41576: SAMZA-831: preparing 0.10.0 release candidate

2015-12-21 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41576/#review111515 --- Ship it! Ship It! - Navina Ramesh On Dec. 19, 2015, 1:24 a.m.,

Re: KafkaSystemProducer partitioning

2015-12-21 Thread Yi Pan
Hi, Michal, Yeah, unfortunately, based on the current code base, the only short term solution would be your proposal.The issue is the following: 1. Kafka producer only applies the specified partition.class to the ProducerRecord key. Samza allows the user to specify a partition key that is differen