My vote: 0 (non-binding)

+1 tag/commit
+1 git vs source zip
+1 No binaries
0 mvn install fails in test
0 Apache rat is not happy
0 dist includes a  _remote.repositories file - this should be removed
+1 LICENSE DISCLAIMER NOTICE

I'm afraid it's not a positive vote from me because the build fails in
test - and the README told me to do "mvn package" (A workaround here
could be to use -DskipTests=true - but I don't know why the test
fails)


Apache rat complains about:
 !????? CONTRIBUTING.md
 !????? .travis.yml
 !????? bin/samza-kryo

These should be added to pom.xml's apache-rat configuration to be
ignored. None of them are worthy of a license header I think.


Your vote email didn't say the hash of the source distro, it is (sha1)
cab5ace6bfff3b70f883db61578fbda847f0fd66
samoa-0.4.0-incubating-source-release.zip

(which matches *.sha1 on dist)





Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.411
sec <<< FAILURE! - in
org.apache.samoa.topology.impl.SimpleEntranceProcessingItemTest
testStartSendingEvents(org.apache.samoa.topology.impl.SimpleEntranceProcessingItemTest)
 Time elapsed: 0.313 sec  <<< ERROR!
java.lang.IllegalStateException: Missing invocation to mocked type at
this point; please make sure there is an associated mock field or mock
parameter in scope
at 
org.apache.samoa.topology.impl.SimpleEntranceProcessingItemTest$4.<init>(SimpleEntranceProcessingItemTest.java:159)
at 
org.apache.samoa.topology.impl.SimpleEntranceProcessingItemTest.testStartSendingEvents(SimpleEntranceProcessingItemTest.java:155)

[INFO] Apache SAMOA ....................................... SUCCESS [  4.748 s]
[INFO] samoa-instances .................................... SUCCESS [  3.676 s]
[INFO] samoa-api .......................................... SUCCESS [ 22.050 s]
[INFO] samoa-test ......................................... SUCCESS [  2.149 s]
[INFO] samoa-local ........................................ FAILURE [ 49.418 s]

using

Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
2015-11-10T16:41:47+00:00)
Maven home: /home/stain/software/maven
Java version: 1.8.0_91, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "4.4.0-38-generic", arch: "amd64", family: "unix"

On 19 September 2016 at 14:50, Nicolas Kourtellis <nkourtel...@gmail.com> wrote:
> Hi all,
>
> Our new release has been voted from the Apache SAMOA team and we are
> opening the vote to the incubator email list for testing.
>
> Please vote on releasing the following release candidate as Apache
> SAMOA (incubating)
> version 0.4.0. This release will be the second release for SAMOA in the
> incubator.
>
> -------------------------------------------------------------
> The commit to be voted on is in the branch "releases/0.4.0-incubating"
> (commit fc39238dd7d3674c069a8142312da8c1812bc907):
> https://git1-us-west.apache.org/repos/asf/incubator-samoa/
> repo?p=incubator-samoa.git;a=commit;h=fc39238dd7d3674c069a8142312da8
> c1812bc907
>
> Tag v0.4.0-incubating:
> https://git1-us-west.apache.org/repos/asf/incubator-samoa/
> repo?p=incubator-samoa.git;a=tag;h=aa5bd941ccbed1aabb46b8119049ac1bb293c3a2
>
> Release artifacts are signed with the following key:
> *https://people.apache.org/keys/committer/nkourtellis.asc
> <https://people.apache.org/keys/committer/nkourtellis.asc>*
>
> The staging repository for this release can be found at:
> https://repository.apache.org/content/repositories/staging/
> org/apache/samoa/samoa/0.4.0-incubating/
>
> The developer's version artifacts:
> https://dist.apache.org/repos/dist/dev/incubator/samoa/0.4.0-incubating-rc1/
>
> -------------------------------------------------------------
>
> Please vote on releasing this package as Apache SAMOA 0.4.0 (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 SAMOA 0.4.0 (incubating)
> [ ] -1 Do not release this package because ...
>
> I'm +1 on the release.
>
> Cheers,
>
> Nicolas
>
>
>
> --
> Nicolas Kourtellis



-- 
Stian Soiland-Reyes
http://orcid.org/0000-0001-9842-9718

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to