[GitHub] accumulo pull request #178: ACCUMULO-4505: Fix for Shell erroneously reading...

2016-11-04 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/178#discussion_r86646752 --- Diff: shell/src/main/java/org/apache/accumulo/shell/Shell.java --- @@ -492,7 +491,8 @@ private static Instance getZooInstance(String instanceName,

[GitHub] accumulo pull request #178: ACCUMULO-4505: Fix for Shell erroneously reading...

2016-11-04 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/178#discussion_r86646837 --- Diff: shell/src/main/java/org/apache/accumulo/shell/ShellOptionsJC.java --- @@ -308,6 +308,13 @@ public ClientConfiguration getClientConfiguration()

Re: My mistake (force push)

2016-11-04 Thread Josh Elser
No problem! Mistakes happen :) Thanks for the quick fix and notification of what happened. Christopher wrote: Accumulators, I accidentally made a big mistake with my authorship information in a recent commit to the accumulo git repository. Git was somehow pointing to somebody else's

My mistake (force push)

2016-11-04 Thread Christopher
Accumulators, I accidentally made a big mistake with my authorship information in a recent commit to the accumulo git repository. Git was somehow pointing to somebody else's .gitconfig file when I authored my commits for ACCUMULO-4514, so the authorship information on my last commits were

[GitHub] accumulo pull request #179: ACCUMULO-4502 Added volative to some class membe...

2016-11-04 Thread ivakegg
GitHub user ivakegg opened a pull request: https://github.com/apache/accumulo/pull/179 ACCUMULO-4502 Added volative to some class members to mitigate issues… … when one thread seeks the iterator, and another thread is reading it. You can merge this pull request into a Git

[GitHub] accumulo pull request #178: ACCUMULO-4505: Fix for Shell erroneously reading...

2016-11-04 Thread milleruntime
GitHub user milleruntime opened a pull request: https://github.com/apache/accumulo/pull/178 ACCUMULO-4505: Fix for Shell erroneously reading accumulo-site.xml Fixes in a couple places for ACCUMULO-4505. Shell was getting the site instance config (which read accumulo-site) when