Re: Review Request 49877: SAMZA-972: Holistic memory monitoring for SamzaContainer

2016-07-13 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49877/ --- (Updated July 14, 2016, 2:37 a.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 49877: SAMZA-972: Holistic memory monitoring for SamzaContainer

2016-07-13 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49877/ --- (Updated July 14, 2016, 2:28 a.m.) Review request for samza, Boris Shkolnik, Ch

Re: Review Request 49877: SAMZA-972: Holistic memory monitoring for SamzaContainer

2016-07-13 Thread Jagadish Venkatraman
> On July 12, 2016, 5:22 p.m., Chris Pettitt wrote: > > samza-core/src/main/java/org/apache/samza/container/host/ProcfsBasedStatisticsMonitor.java, > > line 67 > > > > > > How stable is this across versions of Linux?

Re: Review Request 48213: SAMZA-960: Make system producer thread safe

2016-07-13 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48213/ --- (Updated July 14, 2016, 12:43 a.m.) Review request for samza, Chris Pettitt, Na

Re: Review Request 49877: SAMZA-972: Holistic memory monitoring for SamzaContainer

2016-07-13 Thread Jagadish Venkatraman
> On July 13, 2016, 12:48 a.m., Fred Ji wrote: > > samza-core/src/main/java/org/apache/samza/container/host/ProcfsBasedStatisticsMonitor.java, > > line 63 > > > > > > [Info Question] If only supported in Linux, do yo

Re: Review Request 48356: RFC: Samza as a library

2016-07-13 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48356/ --- (Updated July 13, 2016, 9:58 p.m.) Review request for samza, Chris Pettitt and

Re: Review Request 49877: SAMZA-972: Holistic memory monitoring for SamzaContainer

2016-07-13 Thread Jagadish Venkatraman
> On July 12, 2016, 5:22 p.m., Chris Pettitt wrote: > > Biggest concern with this patch is that it seems to bake in a dependency on > > Linux. Is Samza only supported on Linux? > > > > Other than that, some minor stuff to be fixed, but no major issues. Thanks for the feedback Chris! I thought

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-07-13 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47835/ --- (Updated July 13, 2016, 8:54 a.m.) Review request for samza, Boris Shkolnik, Ch