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

Annoying message while using OpenTelemetry: msg: defaulted common supertype for type <...> due to err:java.lang.IllegalStateException: ClassLoader is null? #2687

Closed
dmitryb-dev opened this issue May 11, 2022 · 3 comments
Assignees
Milestone

Comments

@dmitryb-dev
Copy link

I'm getting a lot of messages like this:

ebean-enhance> cls: io/opentelemetry/javaagent/shaded/io/opentelemetry/context/LazyStorage msg: defaulted common supertype for type1:io/opentelemetry/javaagent/shaded/io/opentelemetry/context/StrictContextStorage type2:io/opentelemetry/javaagent/shaded/io/opentelemetry/context/ContextStorage due to err:java.lang.IllegalStateException: ClassLoader is null?

Probably io.opentelemetry should be added to IgnoreClassHelper.java?

@rbygrave
Copy link
Member

Well, yes we could and probably should add io.opentelemetry to the ignore list - but note that is just a performance feature (ignoring packages that we reasonably know should never contain entity beans, query beans or transactional methods).

It would be good to try and reproduce this issue first it we can. That is, the ClassLoader is null? part more suggests this is more a bug.

Are you able to create a test case? Perhaps a Fork/PR of https://github.com/ebean-orm-examples/example-minimal ?

I am unable to reproduce by for example, adding dependency opentelemetry-sdk and code OpenTelemetrySdk openTelemetrySdk = OpenTelemetrySdk.builder().buildAndRegisterGlobal();

... makes we wonder if there is something interesting with the opentelemetry agent.

@rbygrave
Copy link
Member

Refer to: ebean-orm/ebean-agent#183

@rbygrave rbygrave self-assigned this May 13, 2022
@rbygrave rbygrave added this to the 13.6.1 milestone May 16, 2022
rbygrave added a commit that referenced this issue May 16, 2022
@rbygrave
Copy link
Member

Ok, I'll close this for now as fixed in 13.6.1 with ebean-agent ignoring io.opentelemetry.

If we can get a test case that reproduces with the ClassLoader is null? error we can reopen or create a new one and look at that.

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