Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-08-22 Thread Jake Maes
> On Aug. 23, 2016, 1:21 a.m., Navina Ramesh wrote: > > samza-rest/src/main/java/org/apache/samza/monitor/MonitorLoader.java, line > > 23 > > > > > > I wonder if it will be useful to have this as a more generic Utili

Re: Review Request 50151: SAMZA-975 Initial Samza REST Implementation

2016-08-22 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50151/#review145963 --- Ship it! Overall looks good. Just jotted down some thoughts :)

Re: [DISCUSS] Samza 0.11.0 release

2016-08-22 Thread Nicolas Maquet
Hi all, Would it be possible to consider including the following in the 0.11.0 target release? SAMZA-855: Upgrade Samza's Kafka client version to 0.10.0.0 We are looking at upgrading our Kafka cluster to 0.10 and Samza is currently preventing us from doing so. Regards, Nicolas Maquet 2016-0

[DISCUSS] Samza 0.11.0 release

2016-08-22 Thread xinyu liu
Hi, All, There have been quite a lot of new features added to master since 0.10 release to warrant a new major release. At LinkedIn, we've done functional and performance testing against master in the past weeks, and deployed jobs with the latest build in production. We will continue to test for s

Re: Review Request 51047: SAMZA-1000 Fix hello-samza documentation to not use latest branch by default

2016-08-22 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/#review146424 --- Just noticed that we need to fix one more in the documentation.

Re: Review Request 51228: SAMZA-1000 Update readme for hello-samza with a local build

2016-08-22 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51228/#review146397 --- Fix it, then Ship it! lgtm! README.md (line 8)

Re: Review Request 51047: SAMZA-1000 Fix hello-samza documentation to not use latest branch by default

2016-08-22 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51047/#review146396 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On Aug. 19,

Re: kafka dependency version

2016-08-22 Thread Yi Pan
Hi, Gaurav, There is already an effort going on for this one: SAMZA-855 . It would be good if you can try out the patch. Thanks! -Yi On Mon, Aug 22, 2016 at 1:11 AM, Gaurav Agarwal wrote: > My initial attempt to build against kafka 0.9.0 or 0.1

Re: kafka dependency version

2016-08-22 Thread Gaurav Agarwal
My initial attempt to build against kafka 0.9.0 or 0.10.0 stopped short due to some non-trivial interface changes and refactorings in Kafka; and I did not want to make those changes without having a correct understanding. Can someone guide me what would be the correct procedure to upgrade this dep

kafka dependency version

2016-08-22 Thread Gaurav Agarwal
Hi All, What kafka version is supported by Samza release 0.10.1? In the gradle build file (dependency-versions.gradle) I see it as kafkaVersion = "0.8.2.1". This is very old kafka version and lot of very serious bugs have been fixed since then (including the one we are hitting where kafka client