Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-28 Thread Alex Buck
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/ --- (Updated April 28, 2016, 1:41 p.m.) Review request for samza. Repository: sam

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-28 Thread Alex Buck
> On April 28, 2016, 12:32 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-autoscaling/src/test/java/org/apache/samza/autoscaling/utils/YarnUtilTest.java, > > line 1 > > > > > > Sorry I missed this one in the revi

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

2016-04-28 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44920/#review130945 --- samza-core/src/main/scala/org/apache/samza/metrics/ContainerProce

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-28 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/#review130952 --- Hi, @Alex, thanks for the quick update. There is a few more checks

Re: [Discuss] Moving Samza to Java 1.8 source compatibility.

2016-04-28 Thread Yi Pan
I am +1 on the JDK8 move. As Jake has elaborated, there are numerous advantages from 1.8 source compatible code. As for the downside of dropping JDK7 support, obviously, bin backward-compatibility will be broken. However, moving to JDK8 binary is not a big effort for JDK7-compatible Java and Scala

Re: [Discuss] Moving Samza to Java 1.8 source compatibility.

2016-04-28 Thread Roger Hoover
+1 for me. We're already using Java 8 in PRD. On Thu, Apr 28, 2016 at 10:45 AM, Yi Pan wrote: > I am +1 on the JDK8 move. As Jake has elaborated, there are numerous > advantages from 1.8 source compatible code. > > As for the downside of dropping JDK7 support, obviously, bin > backward-compatib

Re: [Discuss] Moving Samza to Java 1.8 source compatibility.

2016-04-28 Thread Navina Ramesh
+1 for moving to JDK8. We have traditionally been pretty slow at releasing. I think we need to start thinking in terms of long-term release plans and iterate faster. Prior to Samza 1.0, I think we will at least have 2 releases: 0.10.1 -> featuring mostly bug-fixes and improvements to host-affinit