+0 There are issues with rat and binary files. I would like to hear from
others whether these can be ignored in this first release.

Now the details...

The email has all of the necessary information, and the structure of the
zip looks good.

If I understand the maven-release-plugin correctly, the hash of the release
is actually b6af86cb047c8a160981b017dae2e57a8b923122, which is the parent
of 56608ec69bc15a1467ffe689bd613d8daad9f02c.

Checked LICENSE and NOTICE are present in jars.

Checked LICENSE, DISCLAIMER and NOTICE are present in source distribution.

Verified GPG key and hashes.

Builds under JDK 1.7 on ubuntu-15.04. I didn’t try running the test suite,
or JDK 1.8.

Some files have spaces in their names. Not high priority, but consider
removing the spaces. They create havoc for utilities such as “find”.

There are lots of binary files (.snapshot, hbase-export.tar.gz, .svg, …).
And you are distributing tomcat with all its jars (both zipped and
unzipped). And there is kylin.war (i.e. your own binaries).

As a result of those binaries, it is a much larger download than I’d expect
for a source release.

I ran “mvn apache-rat:rat” and it is still running after 10 minutes. Maybe
it is choking on one of those binary files. I ran on the modules
individually:

for i in atopcalcite common metadata dictionary cube job storage query
server jdbc invertedindex; do (cd $i && echo && echo $i && mvn
apache-rat:check); done

and it completed, but there were errors in most modules.

Julian


On May 22, 2015, at 9:31 AM, Luke Han <[email protected]> wrote:

Hi all,

I have created a build for Apache Kylin 0.7.1-incubating, release candidate
1.

Thanks to everyone who has contributed to this release.
This is Apache Kylin first release attempt, please help to verify and let’s
know if there’s any issue.
Here’s release notes:
https://github.com/apache/incubator-kylin/blob/0.7.1-staging/docs/release_notes.md

The commit to be voted upon:
https://github.com/apache/incubator-kylin/commit/56608ec69bc15a1467ffe689bd613d8daad9f02c


Its hash is 56608ec69bc15a1467ffe689bd613d8daad9f02c
<
https://github.com/apache/incubator-kylin/commit/56608ec69bc15a1467ffe689bd613d8daad9f02c
>
.

The artifacts to be voted on are located here:
https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-0.7.1-incubating-rc1/

The hashes of the artifacts are as follows:
src.zip.md5 3995197634625eb6aae35926e9216376
src.zip.sha1 148b95f047d7f3134f6bc68df78d2e1b4a3688dd

A staged Maven repository is available for review at:
https://repository.apache.org/content/repositories/orgapachekylin-1002/

Release artifacts are signed with the following key:
https://people.apache.org/keys/committer/lukehan.asc

Please vote on releasing this package as Apache Kylin 0.7.1-incubating.

The vote is open for the next 72 hours and passes if a majority of
at least three +1 PPMC votes are cast.

[ ] +1 Release this package as Apache Kylin 0.7.1-incubating
[ ]  0 I don't feel strongly about it, but I'm okay with the release
[ ] -1 Do not release this package because...


Here is my vote:

+1 (binding)

Reply via email to