Upgrade to 33.0.0 #12
Conversation
| <argLine>-Duser.timezone=UTC</argLine> | ||
| <argLine> | ||
| -Duser.timezone=UTC | ||
| --add-opens=java.base/sun.nio.ch=ALL-UNNAMED |
There was a problem hiding this comment.
Would we want the full set?
--add-exports=java.base/jdk.internal.misc=ALL-UNNAMED \
--add-exports=java.base/jdk.internal.ref=ALL-UNNAMED \
--add-opens=java.base/java.nio=ALL-UNNAMED \
--add-opens=java.base/sun.nio.ch=ALL-UNNAMED \
--add-opens=java.base/jdk.internal.ref=ALL-UNNAMED \
--add-opens=java.base/java.io=ALL-UNNAMED \
--add-opens=java.base/java.lang=ALL-UNNAMED \
--add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED
There was a problem hiding this comment.
I wanted to remain conservative ... but sure - it can't hurt :)
added them to the pom.xml
pom.xml
Outdated
| <groupId>com.google.guava</groupId> | ||
| <artifactId>guava</artifactId> | ||
| <scope>test</scope> | ||
| </dependency> |
There was a problem hiding this comment.
I had to remove this guava dependency to get the project to build with mvn package. It gets inherited from Druid itself at compile, test, and runtime when removed. But when it's here, it's only present during test. I suppose including it as provided could also work.
There was a problem hiding this comment.
I've also seen some wierd exception - but this is not necessary ; I've removed a few more unnecessary deps
|
another way to do somewhat the same is to use the not sure if that would be a more preferred way for this project.... |
No description provided.