Re: Review Request 44820: SAMZA-896 : Improvements to thread-safety in ContainerRequestState

2016-03-14 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44820/ --- (Updated March 15, 2016, 3:25 a.m.) Review request for samza, Boris Shkolnik, J

Re: Review Request 44820: SAMZA-896 : Improvements to thread-safety in ContainerRequestState

2016-03-14 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44820/ --- (Updated March 15, 2016, 3:19 a.m.) Review request for samza, Boris Shkolnik, J

Re: Review Request 44820: SAMZA-896 : Improvements to thread-safety in ContainerRequestState

2016-03-14 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44820/#review123572 --- Ship it! Ship It! - Jake Maes On March 15, 2016, 1:06 a.m.,

Re: Review Request 44820: SAMZA-896 : Improvements to thread-safety in ContainerRequestState

2016-03-14 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44820/ --- (Updated March 15, 2016, 1:06 a.m.) Review request for samza, Boris Shkolnik, J

Review Request 44820: SAMZA-896 : Improvements to thread-safety in ContainerRequestState

2016-03-14 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44820/ --- Review request for samza. Repository: samza Description --- The Containe

Re: Review Request 44405: SAMZA-882 - Detect partition count changes in input streams

2016-03-14 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44405/#review123527 --- samza-core/src/main/scala/org/apache/samza/coordinator/StreamPart

Re: Review Request 34585: SAMZA-681: create a unit test harness to easily test samza tasks

2016-03-14 Thread George Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34585/#review123510 --- samza-test/src/main/java/org/apache/samza/test/harness/InMemoryTa

Re: Samza with Rocksdb hanged forever in KeyValueStorageEngine.restore()

2016-03-14 Thread Yi Pan
Hi, Weinan, We have not seen this hanging issue at LinkedIn. But it would be good to track that. We will keep an eye on issue-696 as well. Thanks for reporting! -Yi On Sun, Mar 13, 2016 at 8:03 PM, Weinan Zhao wrote: > Hi all, > > I've encountered Samza on yarn couldn't start up and stuck > i

Re: Review Request 44775: SAMZA-894 : Refactor logic that handles allocated containers

2016-03-14 Thread Jagadish Venkatraman
> On March 14, 2016, 6:51 a.m., Navina Ramesh wrote: > > samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerRequestState.java, > > line 149 > > > > > > I remember specifically not simplifying the conditions

Re: Review Request 44775: SAMZA-894 : Refactor logic that handles allocated containers

2016-03-14 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44775/#review123477 --- Ship it! Ship It! - Jake Maes On March 14, 2016, 6:02 a.m.,