-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OAK-9989: introduce wrapped-guava and use it in oak-commons #877
Conversation
oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/FileIOUtils.java
Show resolved
Hide resolved
oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/io/LazyInputStream.java
Show resolved
Hide resolved
oak-commons/src/test/java/org/apache/jackrabbit/oak/commons/FileIOUtilsTest.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still see some instances of the unshaded guava
jar. If not, intentionally, then we should remove them.
@rishabhdaim - thanks for the review. The leftovers in oak-commons are intentional; that is a module that currently has Guava APIs in it's public interfaces. These are deprecated and will be removed in a separate step. (I picked that module for the proof of concept exactly because of the additional complexitiy in this module). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I'm undecided on 'wrapped' vs 'shaded' guava.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
No description provided.