[VOTE] Accumulo 1.7.2-rc2

2016-06-17 Thread Mike Drob
Accumulo Developers, Please consider the following candidate for Accumulo 1.7.2. All content generated via assemble/build.sh --create-release-candidate -P '!thrift' Changes from 1.7.2-rc1 ACCUMULO-4346 correct LICENSE file for source to include text of reference ACCUMULO-4347 Crypto

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Christopher
On Fri, Jun 17, 2016, 22:18 Sean Busbey wrote: > On Fri, Jun 17, 2016 at 5:21 PM, Christopher wrote: > > > > > > Sean, I noticed you committed the change you wanted to the LICENSE files, > > in spite of my reference here indicating (more or less

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Sean Busbey
On Fri, Jun 17, 2016 at 5:21 PM, Christopher wrote: > > > Sean, I noticed you committed the change you wanted to the LICENSE files, > in spite of my reference here indicating (more or less definitively) that > it wasn't actually necessary. The change itself doesn't bother me

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Christopher
On Fri, Jun 17, 2016 at 4:30 PM Christopher wrote: > On Fri, Jun 17, 2016 at 3:28 PM Josh Elser wrote: > >> >> >> Mike Drob wrote: >> > Thanks for taking a look, Sean. >> > >> > The LICENSE file in the source tarball refers to the BSD license and >> >

Re: [RESULT] [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Christopher
I recommend preserving the staging repository until the final release. It's useful to compare between different RCs. On Fri, Jun 17, 2016 at 4:38 PM Mike Drob wrote: > This vote fails with one +1, one -1, and one +1 after the deadline. > > The staging repository will be dropped

Re: [RESULT] [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Mike Drob
This vote fails with one +1, one -1, and one +1 after the deadline. The staging repository will be dropped and I will spin up a new RC later tonight. Thanks, Mike On Fri, Jun 17, 2016 at 3:10 PM, Keith Turner wrote: > +1 > > Just completed 2nd CI test run w/ Agitation and it

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Christopher
On Fri, Jun 17, 2016 at 3:28 PM Josh Elser wrote: > > > Mike Drob wrote: > > Thanks for taking a look, Sean. > > > > The LICENSE file in the source tarball refers to the BSD license and > > includes "for details see > >

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Sean Busbey
On Fri, Jun 17, 2016 at 2:53 PM, Sean Busbey wrote: > > > I'll file JIRAs for both issues, but the first one is still a blocker > for me, though I can understand why other folks might still vote +1. FYI, patches for both issues are now up on ACCUMULO-4346 and ACCUMULO-4347

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Keith Turner
+1 Just completed 2nd CI test run w/ Agitation and it was successful. Ran 2 CI runs for 24 hrs w/ and w/o agitation. Ran on on EC2 w/ 1 m3.xlarge master node and 8 d2.xlarge worker nodes. Used ZK 3.4.8, Hadoop 2.6.3, and Centos 7. Below is relevant output of MR job that verified run w/o

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Sean Busbey
On Fri, Jun 17, 2016 at 2:28 PM, Josh Elser wrote: > > > Mike Drob wrote: >> >> Thanks for taking a look, Sean. >> >> The LICENSE file in the source tarball refers to the BSD license and >> includes "for details see >> core/src/main/java/org/apache/accumulo/core/bloomfilter"

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Josh Elser
Mike Drob wrote: Thanks for taking a look, Sean. The LICENSE file in the source tarball refers to the BSD license and includes "for details see core/src/main/java/org/apache/accumulo/core/bloomfilter" and all files there (BloomFilter.java, DynamicBloomFilter.java, and Filter.java) include the

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Mike Drob
Thanks for taking a look, Sean. The LICENSE file in the source tarball refers to the BSD license and includes "for details see core/src/main/java/org/apache/accumulo/core/bloomfilter" and all files there (BloomFilter.java, DynamicBloomFilter.java, and Filter.java) include the full 3-Clause BSD

Re: [VOTE] Accumulo 1.7.2-rc1

2016-06-17 Thread Sean Busbey
-1 good: * verified checksums and signatures * source artifact corresponds to referenced commit * source builds correctly with Oracle JDK 1.7.0_80 / Apache Maven 3.3.9 (including unit tests, not including ITs) bad: * LICENSE in source tarball references the "3 clause BSD" and "MIT" licenses