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

2016-04-27 Thread Alex Buck
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/ --- Review request for samza. Repository: samza Description --- fix issue wi

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

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

Re: Review Request 37026: SAMZA-727: Support for Kerberos

2016-04-27 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37026/#review130672 --- Thanks, @Chen Song for pulling this off! Overall lgtm. I have a hi

Re: Review Request 46282: SAMZA-928 document Kerberos on YARN

2016-04-27 Thread Yi Pan (Data Infrastructure)
> On April 26, 2016, 7:33 p.m., Yi Pan (Data Infrastructure) wrote: > > docs/learn/documentation/versioned/yarn/yarn-security.md, line 28 > > > > > > nit: remove *config*, we can directly refer to the coordinator stre

Re: Review Request 45258: Abandon producer retry after a certain # of errors : SAMZA-911

2016-04-27 Thread Yi Pan (Data Infrastructure)
> On April 21, 2016, 5:21 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-kafka/src/main/scala/org/apache/samza/system/kafka/KafkaSystemProducer.scala, > > line 44 > > > > > > It would be nice to add a producer c

Re: Review Request 45258: Abandon producer retry after a certain # of errors : SAMZA-911

2016-04-27 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45258/#review130847 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On April 15

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

2016-04-27 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/#review130870 --- Ship it! +1 lgtm. Thanks! - Yi Pan (Data Infrastructure) On

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

2016-04-27 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/#review130871 --- samza-autoscaling/src/test/java/org/apache/samza/autoscaling/util

[Discuss] Moving Samza to Java 1.8 source compatibility.

2016-04-27 Thread Jacob Maes
Hey everyone, I wanted to start a discussion to see what folks think about moving to Java 1.8 source compatibility at some point after the 10.1 release. Java 8 has a number of nice features that can help us build more concise, maintainable, and robust software. A few notable features that would b