incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stian Soiland-Reyes <>
Subject Re: [VOTE] Pulsar 1.22.0-incubating Release Candidate 3
Date Fri, 02 Mar 2018 10:00:03 GMT
?On 25 February 2018 at 23:36, Jai Asher <> wrote:
> This is the fourth release candidate for Apache Pulsar, version
> 1.22.0-incubating.
> It fixes the following issues:
> *** Please download, test and vote by Friday, Feb 23, 2018, 10:00 GMT.

Oo, I'll better hurry up then! :)
Normally the wording is "The vote is open for at least 72 hours" --
you are OK to keep it open a bit longer if you need sufficient votes
or have not tallied them yet.

> Source and binary files:
> pulsar-1.22.0-incubating-candidate-3/

Is it fourth of third release candidate? Use consistent numbering.
Starting with "RC0" is a bit unusual..

> Maven staging repo:

My vote: +1 (binding)

+0 checksums valid (I think)
? asc signatures (sorry, my machine lacks gpg today)
+1 tag vs commit
+0 src.tar.gz vs git tag  ( and missing from dist - are they needed?)
+1 mvn install
+1 src LICENSE
+1 bin NOTICE
+0 src NOTICE -- copyright should extend into 2017-2018
+1 src/bin DISCLAIMER
+1 bin LICENSE and licenses (but why extra LICENSE-HdrHistogram0.txt?)
+0 bin NOTICE -- are all of these copyrights really forwarded from their NOTICE?
+1 mvn apache-rat:check - well-documented excludes

As an overall comment I think good work on the licenses!

I did a spot check, and guava.jar does not have a NOTICE, so unless
that was copied from a zip/tar that had such a NOTICE, then there
would be nothing to propagate. On the other side netty.jar has a
humongous NOTICE which somehow just becomes "Copyright 2014 The Netty
Project" in your NOTICE -- this seems to violate their Apache license.
  Has this been discussed on legael?

Your Git repository contains .gitignore.swp from vim which you
probably want to delete.

Your checksum files are in an unusual style:

C1 B8 C8 91 23 92 6A 56  82 F6 E9 F3 25 86 8B 58

CA1B352F 9576C8CB F16258F8 DEABF8F6 E95A926F 665E2FD8 30A38532 8BC639C6 20FD34E6
 6948396A CCD1A123 F072F93D 55D316EB EE34D208 9E0E9174 95AA09EE

Normally the .md5 and .sha512 files contain the checksum only, in
lowercase hex without spacing, e.g.


This makes it easier to check against tools like md5sum and shasum.

You didn't include .sha1 checksums, but extra points for .sha512 :)

It is customary to include the checksums (at least md5) or the svn revision in the [VOTE] email, to any avoid
accidental last-minute-tampering confusion and to keep it in the
mailing list archives.

Tested with:
Apache Maven 3.5.0 (ff8f5e7444045639af65f6095c62210b5713f426;
Maven home: /usr/local/share/maven
Java version: 1.8.0_144, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_144.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.11.6", arch: "x86_64", family: "mac"

Stian Soiland-Reyes

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message