Re: [VOTE] Apache Beam release 0.3.0-incubating

2016-10-29 Thread Jean-Baptiste Onofré
+1 (binding) Regards JB ⁣​ On Oct 28, 2016, 10:58, at 10:58, Aljoscha Krettek wrote: >Hi everyone, >Please review and vote on the release candidate #1 for the Apache Beam >version 0.3.0-incubating, as follows: >[ ] +1, Approve the release >[ ] -1, Do not approve the

Re: [VOTE] Apache Beam release 0.3.0-incubating

2016-10-28 Thread Jean-Baptiste Onofré
Hi John Rat is supposed to run with the release profile. We are going to check that and why DEPENDENCIES file has not been checked. Regarding Kinesis, the dependency should not be embedded in any Beam jar or distribution. The user has to explicitly define the dependency to be able to use the

Re: [VOTE] Apache Beam release 0.3.0-incubating

2016-10-28 Thread John D. Ament
Hi, mvn apache-rat:check fails on your release due to the DEPENDENCIES file not having a header. If you don't need this file, please remove it. I would also recommend leaving apache-rat running all the time to avoid newly introduced issues. In addition, I notice that your build output includes

[VOTE] Apache Beam release 0.3.0-incubating

2016-10-28 Thread Aljoscha Krettek
Hi everyone, Please review and vote on the release candidate #1 for the Apache Beam version 0.3.0-incubating, as follows: [ ] +1, Approve the release [ ] -1, Do not approve the release (please provide specific comments) The complete staging area is available for your review, which includes: *