[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-08 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r105063169 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-08 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104965493 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-08 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104952630 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-08 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104950540 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104803563 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r10480 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104800761 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104800443 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104797448 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104796515 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104796382 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104793591 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104792661 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104786398 --- Diff: start/src/main/java/org/apache/accumulo/start/classloader/vfs/AccumuloVFSClassLoader.java --- @@ -171,22 +171,10 @@ public void run() {

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104785603 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104785958 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104786227 --- Diff: start/src/main/java/org/apache/accumulo/start/Main.java --- @@ -212,6 +212,7 @@ public static void printUsage() {

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104761530 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104759592 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/230#discussion_r104707884 --- Diff: assemble/bin/accumulo --- @@ -59,43 +59,22 @@ function main() { export ACCUMULO_CONF_DIR="$conf" # Verify setting in

[GitHub] accumulo pull request #230: Accumulo-4599 Simplify how accumulo script adds ...

2017-03-07 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/230 Accumulo-4599 Simplify how accumulo script adds jars to classpath * Removed ACCUMULO_XTRAJARS variable * Include '-add' option in accumulo script usage You can merge this pull request into