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

Warning for unused classes #25

Closed
vlad-bondarenko opened this issue Mar 31, 2018 · 3 comments
Closed

Warning for unused classes #25

vlad-bondarenko opened this issue Mar 31, 2018 · 3 comments

Comments

@vlad-bondarenko
Copy link
Contributor

Hi!

After running my app, output to log:
мар 31, 2018 5:35:31 PM one.nio.serial.Repository addOptionalBootstrap
WARNING: Missing optional bootstrap class: one.app.remote.reflect.MethodId
мар 31, 2018 5:35:31 PM one.nio.serial.Repository addOptionalBootstrap
WARNING: Missing optional bootstrap class: one.app.remote.comp.RemoteMethodCallRequest

I see source code and found row on one.nio.serial.Repository:
addOptionalBootstrap("one.app.remote.reflect.MethodId");
addOptionalBootstrap("one.app.remote.comp.RemoteMethodCallRequest");

This rows is garbage?

Thanks.

@apangin
Copy link
Member

apangin commented Mar 31, 2018

Optional classes belong to our remoting framework which is not yet open source.
I think I'll improve this part of integration so that one-nio users won't see annoying warnings.

@vlad-bondarenko
Copy link
Contributor Author

Thanks.

@apangin
Copy link
Member

apangin commented Jun 14, 2018

Fixed in 992fd09

@apangin apangin closed this as completed Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants