+1 (binding)

First off, interesting use of the fetch.sh script to circumvent any extra licensing in Fluo. I feel like this kind of goes against the intent of the LICENSE and NOTICE files (they're meant to be there so that users know your product depends on and those licenses), but I can't cite a reason why it's against policy.


* xsum/sigs OK
* no binary files in source-release
* apache-rat:check passes
* mvn verify passes (sans GarbageCollectionIteratorIT[1])
* DISCLAIMER is present
* L&N are fine
* KEYS present and contains the signing key
* SHA1 present in repo

* I feel like you could easily add the license header to the docs/**/*.md, but I know I've seen this go both ways. * I would like to see apache-fluo in the tarball names and directories contained in the tarballs, but I've also argued at length with Christopher about this in the past. I will probably continue to cite it as something I'd like to see :) * I would guess some people in IPMC would like to see your artifacats staged to dist.a.o/repos/dist/dev/incubator/fluo but nothing wrong with references from repo.a.o AFAIK

Made a pass over the website and things look OK too. Will be happy to see the two reference to "pre-ASF life" things removed promptly after 1.0.0-incubating is released.


- Josh

[1]

```
Running org.apache.fluo.integration.impl.GarbageCollectionIteratorIT
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 113.007 sec <<< FAILURE! - in org.apache.fluo.integration.impl.GarbageCollectionIteratorIT testGetOldestTimestamp(org.apache.fluo.integration.impl.GarbageCollectionIteratorIT) Time elapsed: 21.67 sec <<< FAILURE!
java.lang.AssertionError: expected:<0> but was:<2>
at org.apache.fluo.integration.impl.GarbageCollectionIteratorIT.testGetOldestTimestamp(GarbageCollectionIteratorIT.java:194)
```

Christopher wrote:
Fluo Developers,

Please consider the following candidate for Fluo 1.0.0-incubating.

Git Commit:
     b07fbf8b755e8f555d06fe2d57db2868f9b20c1c
Branch:
     1.0.0-incubating-rc1

If this vote passes, a gpg-signed tag will be created using:
     git tag -f -m 'Apache Fluo 1.0.0-incubating' -s
rel/fluo-1.0.0-incubating \
     b07fbf8b755e8f555d06fe2d57db2868f9b20c1c

Staging repo:
https://repository.apache.org/content/repositories/orgapachefluo-1012
Source (official release artifact):
https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-source-release.tar.gz
Binary:
https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-bin.tar.gz
(Append ".sha1", ".md5", or ".asc" to download the signature/hash for a
given artifact.)

All artifacts were built and staged with:
     mvn release:prepare&&  mvn release:perform

Signing keys are available at
https://www.apache.org/dist/incubator/fluo/KEYS
(Expected fingerprint: 8CC4F8A2B29C2B040F2B835D6F0CDAE700B6899D)

Release notes (in progress) can be found at:
https://fluo.apache.org/.../1.0.0-incubating

Please vote one of:
[ ] +1 - I have verified and accept...
[ ] +0 - I have reservations, but not strong enough to vote against...
[ ] -1 - Because..., I do not accept...
... these artifacts as the 1.0.0-incubating release of Apache Fluo.

This vote will end on Fri Sep 23 23:30:00 UTC 2016
(Fri Sep 23 19:30:00 EDT 2016 / Fri Sep 23 16:30:00 PDT 2016)

Thanks!

P.S. Hint: download the whole staging repo with
     wget -erobots=off -r -l inf -np -nH \
     https://repository.apache.org/content/repositories/orgapachefluo-1012/
     # note the trailing slash is needed

Reply via email to