Mmuzaf commented on a change in pull request #8406:
URL: https://github.com/apache/ignite/pull/8406#discussion_r518288771
##########
File path: modules/opencensus/pom.xml
##########
@@ -116,4 +116,27 @@
<scope>test</scope>
</dependency>
</dependencies>
+
+ <build>
+ <plugins>
+ <plugin>
+ <artifactId>maven-dependency-plugin</artifactId>
+ <executions>
+ <execution>
+ <id>copy-libs</id>
+ <phase>test-compile</phase>
Review comment:
Can you clarify the following questions, please:
- why phase `test-compile` instead of `package`?
- all the `opencensus` libs will be available at `libs` directory by
default, right? should we copy them from e.g. `libs/opencensus` directory only
when this integration is used?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
