Skip to content
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

ND4J/SameDiff - another "illegal reflective access operation" #8139

Closed
AlexDBlack opened this issue Aug 23, 2019 · 1 comment
Closed

ND4J/SameDiff - another "illegal reflective access operation" #8139

AlexDBlack opened this issue Aug 23, 2019 · 1 comment
Assignees
Labels
ND4J ND4J Issues SameDiff Autodiff related issues
Milestone

Comments

@AlexDBlack
Copy link
Contributor

As per #8123 but for a different library.
Previously it was only printing the first instance by default, so this wasn't visible until now. :/

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.github.os72.protobuf351.UnsafeUtil (file:/C:/Users/Alex/.m2/repository/com/github/os72/protobuf-java-shaded-351/0.9/protobuf-java-shaded-351-0.9.jar) to field java.nio.Buffer.address
WARNING: Please consider reporting this to the maintainers of com.github.os72.protobuf351.UnsafeUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Note it's also JVM-specific, not every JVM will throw this...
AdoptOpenJDK 8.0.222.10-hotspot shows the above, whereas Oracle JDK 1.8.0_201 shows no warning.

@AlexDBlack AlexDBlack added ND4J ND4J Issues SameDiff Autodiff related issues labels Aug 23, 2019
@AlexDBlack AlexDBlack added this to the 1.0.0-beta5 milestone Aug 23, 2019
@AlexDBlack AlexDBlack self-assigned this Aug 23, 2019
@AlexDBlack
Copy link
Contributor Author

This warning is due to ProtoBuf: protocolbuffers/protobuf#3781
Next release after that fix was implemented is 3.7.0.
We use an externally shaded version of protobuf in ND4J (mainly for TF import), but the shaded version is only released up to 3.6.0.
http://central.maven.org/maven2/com/github/os72/
Looks like we'll have to switch to shading protobuf ourselves in order to fix this...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ND4J ND4J Issues SameDiff Autodiff related issues
Projects
None yet
Development

No branches or pull requests

1 participant