incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ryan Blue <rb...@netflix.com.INVALID>
Subject Re: [VOTE] Apache Toree 0.2.0-incubating (RC1)
Date Fri, 03 Nov 2017 17:26:12 GMT
-1 (binding)

There are a few things that could be improved, but my -1 is because the
release tarball doesn’t match the release tag (inclusion of
.example-image), some files are missing the license header, and there is no
mention in the license file of mesos-protobuf that’s included in the
tarball.

rb


Here are my other notes:

The .sha file has a sha512 checksum, which should be in a .sha512 file.
Also (but minor), both .md5 and .sha files have a full path instead of a
relative path:
/Users/lresende/opensource/jupyter/incubator-toree-apache/dist/toree-src/toree-0.2.0-incubating-src.tar.gz

I had to import the key using gpg --recv-keys EFB55DF1. Is there a KEYS
file published for Toree?

The tarball currently unpacks into the current directory, which is unusual
for source tarballs.

RAT checks fail for some files. Here’s the summary:

 !????? .jvmopts
 !????? .example-image
 !????? sparkr-interpreter/src/main/resources/README.md
 !????? index.ipynb
 !????? README.md
 !????? RELEASE_NOTES.md
 !????? etc/pip_install/MANIFEST.in
 !????? etc/.src-release-ignore

We should have license headers in the .md files, and the release process
should ideally use git archive to avoid picking up files from the local
working directory that aren’t part of the tagged release.

This also distributes a few Jars:

Archives:
 + scala-interpreter/src/test/resources/TestJar2.jar
 + scala-interpreter/src/test/resources/ScalaTestJar.jar
 + scala-interpreter/src/test/resources/TestJar.jar
 + kernel/lib/mesos-0.18.1-shaded-protobuf.jar

I think the test Jars are fine, but LICENSE and NOTICE don’t mention
distributing mesos-protobuf.
​

On Wed, Nov 1, 2017 at 6:37 AM, Atri Sharma <atri.jiit@gmail.com> wrote:

> +1
> -- Checked Headers
> -- Checked License
> -- Checked DISCLAIMER and Incubator policies
>
> On Thu, Oct 26, 2017 at 9:37 PM, Luciano Resende <luckbr1975@gmail.com>
> wrote:
> > Please vote to approve the release of Apache Toree 0.2.0-incubating
> (RC1).
> >
> > The PPM vote thread:
> > https://www.mail-archive.com/dev@toree.incubator.apache.
> org/msg01527.html
> >
> > And the result:
> > https://www.mail-archive.com/dev@toree.incubator.apache.
> org/msg01539.html
> >
> > Tag: v0.2.0-incubating-rc1 (01cd97e9bad04878a8014016c154a50e2a00f21d)
> >
> > https://github.com/apache/incubator-toree/tree/v0.2.0-incubating-rc1
> >
> > All distribution packages, including signatures, digests, etc. can be
> found
> > at:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/toree/0.2.
> 0-incubating-rc1/
> >
> > Staging artifacts can be found at:
> >
> > https://repository.apache.org/content/repositories/orgapachetoree-1007
> >
> > The vote is open for at least 72 hours and passes if a majority of at
> least
> > 3 +1 PMC votes are cast.
> >
> > [ ] +1 Release this package as Apache Toree 0.2.0-incubating
> > [ ] -1 Do not release this package because ...
> >
> > --
> > Luciano Resende
> > http://twitter.com/lresende1975
> > http://lresende.blogspot.com/
>
>
>
> --
> Regards,
>
> Atri
> l'apprenant
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>


-- 
Ryan Blue
Software Engineer
Netflix

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message