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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
