Re: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Julian Hyde
> On Aug 1, 2016, at 5:24 PM, Navina Ramesh > wrote: > > Are you referring to the Gemfile.lock in docs/ ? I do see that in the > source release. I was mistaken. Apparently it is OK to check in Gemfile.lock; see

Re: Review Request 50682: SAMZA-993 Fix logging bug for some scala versions

2016-08-01 Thread Fred Ji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50682/#review144420 --- Ship it! Ship It! - Fred Ji On Aug. 2, 2016, 12:13 a.m.,

Re: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Navina Ramesh
Ok. I added my keys to the KEYS file. @Julian: Thanks for the suggestion. I don't know the reason for choosing home.apache.org as opposed to dist.apache.org. The latter does make more sense. We will update the release procedure for 0.11.0. Are you referring to the Gemfile.lock in docs/ ? I do

Re: Review Request 50682: SAMZA-993 Fix logging bug for some scala versions

2016-08-01 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50682/#review144417 --- Ship it! +1. thanks for the quick fix! - Xinyu Liu On Aug.

Re: Review Request 50682: SAMZA-993 Fix logging bug for some scala versions

2016-08-01 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50682/#review144414 --- Ship it! Ship It! - Navina Ramesh On Aug. 2, 2016, 12:13

Review Request 50682: SAMZA-993 Fix logging bug for some scala versions

2016-08-01 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50682/ --- Review request for samza, Boris Shkolnik, Chris Pettitt, Fred Ji, Jake Maes,

Re: Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/ --- (Updated Aug. 1, 2016, 11:29 p.m.) Review request for samza, Yi Pan (Data

Re: Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/#review144393 --- Ship it! +1 - Xinyu Liu On Aug. 1, 2016, 10:21 p.m.,

Re: Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/#review144391 ---

Re: Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/ --- (Updated Aug. 1, 2016, 10:21 p.m.) Review request for samza, Yi Pan (Data

Re: Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/#review144387 ---

Review Request 50670: SAMZA-991: Continue to report SamzaAppMasterMetrics

2016-08-01 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50670/ --- Review request for samza, Yi Pan (Data Infrastructure), Navina Ramesh, and

Re: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Julian Hyde
+1 (non-binding) Downloaded, built, ran tests, checked L Instructions on installing gradle were nice and clear. I ran rat (I presume that ‘./gradlew rat’ is the correct command) and it passed. I believe that best practice is to put the release bits under

RE: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Garry Turkington
This link seems to say the KEYS file needs be in the distribution directory on the Apache master site but not necessarily inside each released artifact. Which suggests the release process should ensure the signing key is in the publically accessible KEYS file. I guess its then a question of if

RE: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Garry Turkington
+1 (binding) Built the source ran all tests and tried a sample test job. All looks good. One question - the key used to sign isn't in the KEYS file in the git repo -- is this a convenieence mechanism or does the release process mean it should be there? Garry -Original Message- From:

Re: Review Request 50667: SAMZA-989 - Update hello-samza to use the startup logger

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

Review Request 50667: SAMZA-989 - Update hello-samza to use the startup logger

2016-08-01 Thread Jake Maes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50667/ --- Review request for samza, Boris Shkolnik, Chris Pettitt, Fred Ji, Jake Maes,

Re: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Jacob Maes
+1 non-binding Lots of good fixes in this release. On Mon, Aug 1, 2016 at 11:41 AM, Jagadish Venkatraman < jagadish1...@gmail.com> wrote: > Thanks for driving this release.+1 approve (non-binding) > > Thanks, > Jagadish > > On Mon, Aug 1, 2016 at 11:39 AM, Navina Ramesh >

Re: [DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread xinyu liu
+1 (non-binding). Thanks, Xinyu On Mon, Aug 1, 2016 at 11:41 AM, Jagadish Venkatraman < jagadish1...@gmail.com> wrote: > Thanks for driving this release.+1 approve (non-binding) > > Thanks, > Jagadish > > On Mon, Aug 1, 2016 at 11:39 AM, Navina Ramesh > > wrote:

[DISCUSS] [VOTE] Apache Samza 0.10.1 RC0

2016-08-01 Thread Navina Ramesh
Hey all, This is a call for a vote on a release of Apache Samza 0.10.0. Thanks to everyone who has contributed to this release. We are very glad to see some new contributors in this release. **NOTE**: This release is primarily a bug-fix release with no major changes to the public api. Since we

Re: Review Request 50583: SAMZA-954 Improve logging for Samza

2016-08-01 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50583/#review144369 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On July

Re: Review Request 50317: SAMZA-978: update md files to resolve inconsistent links, broken links and some confusing sentences

2016-08-01 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50317/#review144365 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On July