Skip to content

[wip] getModuleGroup removal using single CL for all modules#18859

Draft
SylvainJuge wants to merge 7 commits into
open-telemetry:mainfrom
SylvainJuge:module-group-2
Draft

[wip] getModuleGroup removal using single CL for all modules#18859
SylvainJuge wants to merge 7 commits into
open-telemetry:mainfrom
SylvainJuge:module-group-2

Conversation

@SylvainJuge
Copy link
Copy Markdown
Contributor

Alternative to #18707 by using a single shared classloader for all instrumentation modules.

Hypotheses to verify

  • we should not get any shared class issues as they are loaded exactly once in the same CL (tests should pass as they currently do)
  • we loose the isolation between instrumentation modules
    • for internal instrumentation this is probably not an issue as there is no overlap in classes.
    • for extensions, they are currently isolated by default, so we need to evaluate how likely it could cause issues.

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.

1 participant