As double check, I would like to recall this vote release, the majority reason is the source shouldn't contains binary file like Tomcat.
I will re-generate package and raise vote again. Thanks. Best Regards! --------------------- Luke Han 2015-05-24 23:16 GMT+08:00 Luke Han <[email protected]>: > Thanks Julian. I wil double check the RAT result. > > Some binary files are required for testing and UI, like the mini-cluster > file for unit test, if there's no conflict with Apache Policy, I would like > to continue to include them in the release, so that everyone could download > and run unit test successfully, otherwise they have to download from > somewhere first. > > @All, please help to verify and check, and leave your comments here. > > I can generate RC2 if some changes should to be applied before release. > > Thank you very much. > > > > > Best Regards! > --------------------- > > Luke Han > > 2015-05-23 2:53 GMT+08:00 Julian Hyde <[email protected]>: > >> +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) >> > >
