JiriOndrusek commented on a change in pull request #1902:
URL: https://github.com/apache/camel-quarkus/pull/1902#discussion_r503811898
##########
File path: extensions/leveldb/runtime/pom.xml
##########
@@ -56,6 +57,35 @@
<dependency>
<groupId>org.apache.camel</groupId>
<artifactId>camel-leveldb</artifactId>
+ <exclusions>
+ <exclusion>
+ <artifactId>leveldbjni-all</artifactId>
+ <groupId>org.fusesource.leveldbjni</groupId>
+ </exclusion>
+ </exclusions>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.camel</groupId>
+ <artifactId>camel-leveldb</artifactId>
+ <exclusions>
+ <exclusion>
+ <groupId>org.fusesource.leveldbjni</groupId>
+ <artifactId>*</artifactId>
+ </exclusion>
Review comment:
@ppalaga There is only one question. Current PR removes dependency with
C++ implementation of leveldDB.
Which makes JVM also using java implementation.
I can return dependency back, but it makes it use by native too, which makes
native artifact a little bit bigger.
Do you know about some correct way how to define dependency only for JVM?
(which would be the best solution here)
----------------------------------------------------------------
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]
