Re: Review Request 44920: SAMZA-680 Refactor the Samza AppMaster to support other cluster managers

2016-04-04 Thread Jagadish Venkatraman
> On March 31, 2016, 10:43 p.m., Yi Pan (Data Infrastructure) wrote: > > samza-core/src/main/java/org/apache/samza/clustermanager/ClusterBasedJobCoordinator.java, > > line 61 > > > > > > I think that this is the impl

Re: Review Request 44920: SAMZA-680 Refactor the Samza AppMaster to support other cluster managers

2016-04-04 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/ --- (Updated April 5, 2016, 12:35 a.m.) Review request for samza, Boris Shkolnik, C

Re: Samza 0.10.0 with Kafka 0.9.0.0

2016-04-04 Thread Yi Pan
Hi, Krishna, I just replied to Nick's question to dev list. Let me know if it makes senses to you or not. Thanks! -Yi On Mon, Apr 4, 2016 at 10:18 AM, Krishna wrote: > Hi Yi, > > Any update on Kafka 0.9 move on Samza 0.10.0 ? > > Thanks > > Krishna >

Re: Kafka 0.9 as part of Samza 0.10?

2016-04-04 Thread Yi Pan
Hi, Nick, Thanks for digging out the details from KAFKA JIRAs! I appreciated it! As for upgrading to Kafka 0.9 to fix those critical issues, I am totally w/ you. The discussion on whether Samza 0.10.1 should include Kafka 0.9 fixes or not has just started (by your thread :)). So, we are happy to

Re: Review Request 45144: SAMZA-906 Host Affinity - Minimize task reassignment when container count changes

2016-04-04 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45144/ --- (Updated April 4, 2016, 9:15 p.m.) Review request for samza, Navina Ramesh, Jag

Re: Review Request 45388: SAMZA-919 - Samza - Add milliseconds and threadname to log4j config. Also switch DailyRollingFileAppender

2016-04-04 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45388/#review126940 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On March 28

Re: Review Request 44920: SAMZA-881

2016-04-04 Thread Jagadish Venkatraman
> On March 31, 2016, 10:43 p.m., Yi Pan (Data Infrastructure) wrote: > > I have some high level questions w/ the current class layout/hierarchy. > > Will sync up w/ Jagadish in person. Thanks for the great feedback :-) I'll update the revised patch soon. Publishing this, so that I don't loose

Re: Samza 0.10.0 with Kafka 0.9.0.0

2016-04-04 Thread Krishna
Hi Yi, Any update on Kafka 0.9 move on Samza 0.10.0 ? Thanks Krishna