[GitHub] [fluo] ctubbsii commented on a diff in pull request #1122: WIP getting fluo to build against Accumulo 2.1.0

[GitBox <gi...@apache.org>]

ctubbsii commented on code in PR #1122:
URL: https://github.com/apache/fluo/pull/1122#discussion_r1014560212


##########
pom.xml:
##########
@@ -308,6 +313,11 @@
               <artifactId>hadoop-client-runtime</artifactId>
               <version>${hadoop.version}</version>
             </dependency>
+            <dependency>

Review Comment:
   You have to change it at https://github.com/apache/fluo/blob/9e254e2ed69a4991e59c75801b144e1ce964360f/modules/integration-tests/pom.xml#L28-L29
   
   The version set in the main pom is the version that Fluo shades for itself, but the version set in the integration-tests module is expected to be set to the same version as what is being used by the accumulo2-maven-plugin for the version of Accumulo it is supposed to run.
   
   When I set the version there to also be 0.17, the ITs make progress, but I think there might be something wrong with the logging config, because they are very very very noisy on the console. This could have something to do with the switch to log4j2. It could also have something to do with slf4j2 or the fact that ZK uses logback by default for its slf4j binding, or something else along those lines.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@fluo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org