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

Fix bug of classnotfound at Fuse runtime #18039

Merged
merged 2 commits into from
Aug 23, 2023

Conversation

lucyge2022
Copy link
Contributor

What changes are proposed in this pull request?

fix the bug where fuse has the bug of "java.lang.NoClassDefFoundError: Could not initialize class io.vertx.core.dns.AddressResolverOptions" after jetcd dependency was introduced

Why are the changes needed?

classes from native side, if not explicitly loaded, it will spew "NoClassDefFoundError" at runtime when the relevant classes was invoked.

Does this PR introduce any user facing changes?

No.

@lucyge2022 lucyge2022 requested a review from LuQQiu August 22, 2023 00:25
@lucyge2022 lucyge2022 changed the title Fuse bug fix Fix bug of classnotfound at Fuse runtime Aug 22, 2023
@yyongycy
Copy link
Contributor

just curious what causes this class special? Need explicit load.

@lucyge2022
Copy link
Contributor Author

just curious what causes this class special? Need explicit load.

this class is used by jetcd client internally to start a grpc channel, and for some reason native couldn't find it unless a explicit call to say JNIEnv->FindClass() is made, I've tried doing ClassLoader at AlluxioFuse side to just load the class, but still it couldn't find in native. One another way is to create an instance of this class at time of starting AlluxioFuse, but I thought its not extendable if there's other places using native lib, but as we are explicitly checking in .dylib and .so lib let me try if I can load the class at process startup time.

@lucyge2022 lucyge2022 added the type-bug This issue is about a bug label Aug 22, 2023
@lucyge2022
Copy link
Contributor Author

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 5e655a0 into Alluxio:main Aug 23, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug This issue is about a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants