Agree. We already discussed about that on the mailing list. I mentionned this some weeks ago.
Regards JB On Oct 26, 2016, 02:26, at 02:26, Dan Halperin <dhalp...@google.com.INVALID> wrote: >My reading of the LEGAL threads is that since we are not including >(shading >or bundling) the ASL-licensed code we are fine to distribute kinesis-io >module. This was the original conclusion that LEGAL-198 got to, and >that >thread has not been resolved differently (even if Spark went ahead and >broke the assembly). The beam-sdks-java-io-kinesis module is an >optional >part (Beam materially works just fine without it). > >So I think we're fine to keep this vote open. > >+1 (binding) on the release > >Thanks Aljoscha! > > >On Tue, Oct 25, 2016 at 12:07 PM, Aljoscha Krettek ><aljos...@apache.org> >wrote: > >> Yep, I was looking at those same threads when I reviewing the >artefacts. >> The release was already close to being finished so I went through >with it >> but if we think it's not good to have them in we should quickly >cancel in >> favour of a new RC without a published Kinesis connector. >> >> On Tue, 25 Oct 2016 at 20:46 Dan Halperin ><dhalp...@google.com.invalid> >> wrote: >> >> > I can't tell whether it is a problem that we are distributing the >> > beam-sdks-java-io-kinesis module [0]. >> > >> > Here is the dev@ discussion thread [1] and the (unanswered) >relevant >> LEGAL >> > thread [2]. >> > We linked through to a Spark-related discussion [3], and here is >how to >> > disable distribution of the KinesisIO module [4]. >> > >> > [0] >> > >> > https://repository.apache.org/content/repositories/staging/ >> org/apache/beam/beam-sdks-java-io-kinesis/ >> > [1] >> > >> > https://lists.apache.org/thread.html/6784bc005f329d93fd59d0f8759ed4 >> 745e72f105e39d869e094d9645@%3Cdev.beam.apache.org%3E >> > [2] >> > >> > https://issues.apache.org/jira/browse/LEGAL-198? >> focusedCommentId=15471529&page=com.atlassian.jira. >> plugin.system.issuetabpanels:comment-tabpanel#comment-15471529 >> > [3] https://issues.apache.org/jira/browse/SPARK-17418 >> > [4] https://github.com/apache/spark/pull/15167/files >> > >> > Dan >> > >> > On Tue, Oct 25, 2016 at 11:01 AM, Seetharam Venkatesh < >> > venkat...@innerzeal.com> wrote: >> > >> > > +1 >> > > >> > > Thanks! >> > > >> > > On Mon, Oct 24, 2016 at 2:30 PM Aljoscha Krettek ><aljos...@apache.org> >> > > wrote: >> > > >> > > > Hi Team! >> > > > >> > > > Please review and vote at your leisure on release candidate #1 >for >> > > 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: >> > > > * JIRA release notes [1], >> > > > * the official Apache source release to be deployed to >> dist.apache.org >> > > > [2], >> > > > * all artifacts to be deployed to the Maven Central Repository >[3], >> > > > * source code tag "v0.3.0-incubating-RC1" [4], >> > > > * website pull request listing the release and publishing the >API >> > > reference >> > > > manual [5]. >> > > > >> > > > Please keep in mind that this release is not focused on >providing new >> > > > functionality. We want to refine the release process and make >stable >> > > source >> > > > and binary artefacts available to our users. >> > > > >> > > > The vote will be open for at least 72 hours. It is adopted by >> majority >> > > > approval, with at least 3 PPMC affirmative votes. >> > > > >> > > > Cheers, >> > > > Aljoscha >> > > > >> > > > [1] >> > > > >> > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa? >> > > projectId=12319527&version=12338051 >> > > > [2] >> > > > >> > >https://dist.apache.org/repos/dist/dev/incubator/beam/0.3.0-incubating/ >> > > > [3] >> > > > https://repository.apache.org/content/repositories/staging/ >> > > org/apache/beam/ >> > > > [4] >> > > > >> > > > https://git-wip-us.apache.org/repos/asf?p=incubator-beam. >> git;a=tag;h= >> > > 5d86ff7f04862444c266142b0d5acecb5a6b7144 >> > > > [5] https://github.com/apache/incubator-beam-site/pull/52 >> > > > >> > > >> > >>
