[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104566494 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummarySerializer.java --- @@ -0,0 +1,543 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104566230 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java --- @@ -117,6 +121,7 @@ String getFormatDescription() { * @return

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104520129 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummarySerializer.java --- @@ -0,0 +1,543 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104519924 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummarySerializer.java --- @@ -0,0 +1,543 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104515984 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummaryInfo.java --- @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104508413 --- Diff: core/src/main/java/org/apache/accumulo/core/client/summary/summarizers/DeletesSummarizer.java --- @@ -0,0 +1,75 @@ +/* + * Licensed

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104563673 --- Diff: docs/src/main/asciidoc/chapters/summaries.txt --- @@ -0,0 +1,211 @@ +// Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104510474 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java --- @@ -117,6 +121,7 @@ String getFormatDescription() { * @return

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104511568 --- Diff: core/src/main/java/org/apache/accumulo/core/metadata/schema/MetadataScanner.java --- @@ -0,0 +1,228 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104511336 --- Diff: core/src/main/java/org/apache/accumulo/core/metadata/schema/MetadataScanner.java --- @@ -0,0 +1,228 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104512326 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummarizerConfigurationUtil.java --- @@ -0,0 +1,128 @@ +/* + * Licensed to the

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104530943 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummarySerializer.java --- @@ -0,0 +1,543 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104516639 --- Diff: core/src/main/java/org/apache/accumulo/core/summary/SummaryReader.java --- @@ -0,0 +1,236 @@ +/* + * Licensed to the Apache Software

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Christopher
Sorry, meant to comment. The close this morning that you did seems to have failed due to it not being able to access some public keyservers. Probably a temporary network issue. I created https://issues.apache.org/jira/browse/INFRA-13620 and closed the repo successfully this afternoon. On Mon, Mar

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Josh Elser
Looks good for me now (whereas it didn't when I emailed earlier). Thanks, Ed! Ed Coleman wrote: My keys show up at pgp.mit.edu (https://pgp.mit.edu/pks/lookup?op=vindex=0xFC4368E0864BCC36) Is there something else I need to do? I closed the staging repository this morning - and it is

RE: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Ed Coleman
My keys show up at pgp.mit.edu (https://pgp.mit.edu/pks/lookup?op=vindex=0xFC4368E0864BCC36) Is there something else I need to do? I closed the staging repository this morning - and it is showing closed now - has this resolved for you? Ed Coleman -Original Message- From: Michael

[GitHub] accumulo pull request #229: ACCUMULO-4597 fixed bug in rfile-info

2017-03-06 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/accumulo/pull/229 ACCUMULO-4597 fixed bug in rfile-info You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/accumulo ACCUMULO-4597 Alternatively

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Michael Wall
Event: Failed: Signature Validation Monday, March 06, 2017 07:08:37 EST (GMT-0500) typeId signature-staging failureMessage No public key: Key with id: (fc4368e0864bcc36) was not able to be located on http://gpg-keyserver.de/. Upload your public key and try the operation again. failureMessage No

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Christopher
Looks published now, to one server at least: https://pgp.mit.edu/pks/lookup?search=edcoleman%40apache.org=index Where does Nexus require it to be published? On Mon, Mar 6, 2017 at 3:21 PM Josh Elser wrote: > Looks like this was attempted, but Ed just needs to publish his

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/228#discussion_r104537251 --- Diff: minicluster/src/main/java/org/apache/accumulo/cluster/standalone/StandaloneClusterControl.java --- @@ -149,9 +141,8 @@ String

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/228#discussion_r104535648 --- Diff: minicluster/src/main/java/org/apache/accumulo/cluster/standalone/StandaloneAccumuloCluster.java --- @@ -57,7 +57,7 @@ private

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/228#discussion_r104527998 --- Diff: minicluster/src/main/java/org/apache/accumulo/cluster/standalone/StandaloneAccumuloCluster.java --- @@ -57,7 +57,7 @@ private

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/228#discussion_r104528620 --- Diff: minicluster/src/main/java/org/apache/accumulo/cluster/standalone/StandaloneClusterControl.java --- @@ -149,9 +141,8 @@ String

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/228#discussion_r104528417 --- Diff: minicluster/src/main/java/org/apache/accumulo/cluster/standalone/StandaloneClusterControl.java --- @@ -123,11 +114,12 @@ public int

[GitHub] accumulo issue #228: ACCUMULO-4596 Improvements to standalone cluster testin...

2017-03-06 Thread mikewalch
Github user mikewalch commented on the issue: https://github.com/apache/accumulo/pull/228 I tested these changes by running `ReadWriteIT` against and a local cluster started by uno. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] accumulo pull request #228: ACCUMULO-4596 Improvements to standalone cluster...

2017-03-06 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/228 ACCUMULO-4596 Improvements to standalone cluster testing * Limited use of bash env variables in standalone cluster testing * Fixed standalone cluster testing sow that it work with Accumulo

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Josh Elser
Looks like this was attempted, but Ed just needs to publish his GPG key (the close op failed) :) Christopher wrote: Staging repository just needs to be closed. On Mon, Mar 6, 2017, 11:00 Mike Miller wrote: Hey Ed - I am getting this error when clicking the source

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/accumulo/pull/227 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r104463134 --- Diff: core/src/main/java/org/apache/accumulo/core/client/summary/Summary.java --- @@ -0,0 +1,127 @@ +/* + * Licensed to the Apache

[GitHub] accumulo issue #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-06 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/accumulo/pull/224 > @keith-turner we talked about this yesterday, but I wanted to post it here. What would happen if a file is deleted, like maybe compacted and gc'd, after the file list is grabbed?

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/227#discussion_r104456123 --- Diff: assemble/bin/accumulo-util --- @@ -571,20 +570,17 @@ function load_jars_hdfs() { "$HADOOP_PREFIX/bin/hadoop" fs -rm

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Christopher
Staging repository just needs to be closed. On Mon, Mar 6, 2017, 11:00 Mike Miller wrote: > Hey Ed - I am getting this error when clicking the source or binary links: > Repository "orgapacheaccumulo-1064 (staging: open)" > [id=orgapacheaccumulo-1064] exists but is not

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/227#discussion_r104454813 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/NativeMap.java --- @@ -60,39 +60,44 @@ * would be a mistake for long lived

[GitHub] accumulo issue #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for native li...

2017-03-06 Thread mikewalch
Github user mikewalch commented on the issue: https://github.com/apache/accumulo/pull/227 I am going to squash and merge today if no one objects. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

Re: [VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Mike Miller
Hey Ed - I am getting this error when clicking the source or binary links: Repository "orgapacheaccumulo-1064 (staging: open)" [id=orgapacheaccumulo-1064] exists but is not exposed. On Mon, Mar 6, 2017 at 7:11 AM, Ed Coleman wrote: > Accumulo Developers, > > Please consider

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/227#discussion_r104438882 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/NativeMap.java --- @@ -60,39 +60,44 @@ * would be a mistake for long lived

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/227#discussion_r104434464 --- Diff: assemble/bin/accumulo-util --- @@ -411,12 +411,16 @@ function create_config() { } function build_native() { -

[GitHub] accumulo pull request #227: ACCUMULO-4596 Stopped using ACCUMULO_HOME for na...

2017-03-06 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/227#discussion_r104434490 --- Diff: assemble/bin/accumulo-util --- @@ -411,12 +411,16 @@ function create_config() { } function build_native() { -

[VOTE] Accumulo 1.7.3-rc1

2017-03-06 Thread Ed Coleman
Accumulo Developers, Please consider the following candidate for Accumulo 1.7.3. (This branch is based on the same commit as the pre-release rc0 test candidate, commit c3a0d1d5e2f810ebcda346977009ab0fd1bd34f7) Release notes, java docs and other non-code artifacts are still in-progress and not