Skip to content

Add api for exposing helper classes to app class loader#17765

Merged
trask merged 4 commits intoopen-telemetry:mainfrom
laurit:expose-classes
Apr 17, 2026
Merged

Add api for exposing helper classes to app class loader#17765
trask merged 4 commits intoopen-telemetry:mainfrom
laurit:expose-classes

Conversation

@laurit
Copy link
Copy Markdown
Contributor

@laurit laurit commented Apr 9, 2026

Let application class loader load specified classes from instrumentation class loader. Currently we create a proxy class that delegates to the helper class in instrumentation class loader. While the proxy classes work I feel it is more complicated than necessary. There are also some obscure limitations like how do you proxy a class that has a super class or implements interfaces that are also helper classes (not visible from application class loader) or what if the proxied type implements methods that either return or accept helper classes. That being said there is one area where proxies work but the proposed solution does not, that is instrumentations that run in boot loader. There is currently 1 instrumentation remaining that uses a proxy and that instrumentation runs in boot loader.

@laurit laurit requested a review from a team as a code owner April 9, 2026 11:22
@laurit laurit requested a review from SylvainJuge April 9, 2026 11:22
@trask trask added this to the v2.27.0 milestone Apr 17, 2026
@trask trask enabled auto-merge (squash) April 17, 2026 15:35
@trask trask merged commit 164e3c8 into open-telemetry:main Apr 17, 2026
92 of 93 checks passed
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

Successfully merging this pull request may close these issues.

3 participants