accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Josh Elser <josh.el...@gmail.com>
Subject Re: [VOTE] Accumulo 1.6.0-RC4
Date Mon, 28 Apr 2014 16:58:36 GMT


On 4/28/14, 12:20 PM, Mike Drob wrote:
> -1
>
> The good:
>
> * Verified all signatures and checksums.
> * Ran continuous ingest with binary artifact + custom built native maps.
>
>
> The issues, but not enough to vote against:
>
> * Encountered ACCUMULO-2741.
> * Encountered ACCUMULO-2742.
> * Source artifact missing .gitignore
> ** This has been discussed, and I'm voting for precedent here. We can agree
> to disagree, and if this vote passes then a new precedent will have been
> set.
>
>
> The bad:
>
> * CHANGES file contains changes for 1.5.0 and 1.4.0 (BAD)
> ** Past discussion here:http://markmail.org/message/ulvovup36uaa2cav
> ** It seems like we agreed to only include changes from the current major
> release line, but that is not 100% clear.
>
> * Missing licence headers:
> ** README
> ** conf/examples/crypto/readme.txt
> ** test/compat/japi-compliance/README
> ** test/system/continuous/ScaleTest.odp
> **
> docs/src/main/latex/common/state_diagrams/HDFS_WAL_states.odg
>
> **
> docs/src/main/latex/common/state_diagrams/HDFS_WAL_states.pdf
>
> **
> docs/src/main/latex/common/state_diagrams/tablet_states.odg
>
> ** docs/src/main/latex/common/state_diagrams/tablet_states.pdf

Should we be trying to get a license included in a non-plaintext file? 
(odg, pdf, and odp)

I remember that ScaleTest presentation being an issue before we had the 
rat-check automated (correctly) in the build. I haven't seen it since we 
got the *apache*-rat-plugin configured.

Mime
View raw message