[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-25 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170478654 ## File path: maven-plugin/src/it/plugin-test/pom.xml ## @@ -154,4 +150,58 @@ + + +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-25 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170478991 ## File path: pom.xml ## @@ -899,10 +920,30 @@ java17 1.0

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-25 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170478585 ## File path: examples/simple/pom.xml ## @@ -105,4 +101,58 @@ + + + hadoop-def

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170412238 ## File path: pom.xml ## @@ -128,7 +128,9 @@ 3.0.5 1 +2 Review comment: There is

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411383 ## File path: core/pom.xml ## @@ -282,5 +277,50 @@ + + hadoop2 + +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411239 ## File path: assemble/bin/bootstrap_config.sh ## @@ -383,6 +394,25 @@ elif [[ "${HADOOP_VERSION}" == "IOP4

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411152 ## File path: .travis.yml ## @@ -34,4 +34,6 @@ jdk: before_script: - rm -rf $HOME/.m2/repository/org/ap

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411664 ## File path: assemble/pom.xml ## @@ -323,5 +323,67 @@ + + hadoop2 +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411518 ## File path: minicluster/pom.xml ## @@ -154,4 +145,51 @@ + + + hadoop2 +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411608 ## File path: pom.xml ## @@ -128,7 +128,9 @@ 3.0.5 1 +2 Review comment: I think t

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411271 ## File path: assemble/pom.xml ## @@ -256,6 +234,28 @@ + +org.a

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411383 ## File path: core/pom.xml ## @@ -282,5 +277,50 @@ + + hadoop2 + +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170411239 ## File path: assemble/bin/bootstrap_config.sh ## @@ -383,6 +394,25 @@ elif [[ "${HADOOP_VERSION}" == "IOP4

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170385390 ## File path: server/gc/pom.xml ## @@ -87,4 +83,40 @@ test + + + hadoop2 +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170382302 ## File path: assemble/bin/bootstrap_config.sh ## @@ -24,7 +24,7 @@ where options include (long options not

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170384404 ## File path: assemble/pom.xml ## @@ -323,5 +283,147 @@ + + hadoop2 +

[GitHub] ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars

2018-02-23 Thread GitBox
ctubbsii commented on a change in pull request #332: ACCUMULO-4826 Use the hadoop3 shaded jars URL: https://github.com/apache/accumulo/pull/332#discussion_r170384652 ## File path: core/pom.xml ## @@ -282,5 +277,50 @@ + + hadoop2 + +