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

[GR-47365] Integrate proxy registration to the reflection config #8822

Merged
merged 1 commit into from May 22, 2024

Conversation

graalvmbot
Copy link
Collaborator

This PR includes the registration of proxy classes for runtime instantiation into the reflection configuration. This allows the registration of reflection queries on proxy classes, which was not possible until now.
The existing handling of proxy configuration files is still here to handle existing files, but the agent is modified to use the new behavior.

The new format to specify proxy classes is the following:

{
  "type": { "proxy": [
    "InterfaceName1",
    "InterfaceName2",
    ...
  ]},
  ... (member registrations)
}

Here's a summary of the main changes:

  • The agent (BreakpointInterceptor) is modified to consistently handle proxy classes, instead of the case-by-case approach used until now. Proxies are encoded as an array of interfaces in the trace files;
  • ProxyConfigurationTypeDescriptor enables handling of proxy classes in configuration code;
  • Types are now passed as ConfigurationTypeDescriptor from parsing onwards, notably in ConfigurationType;
  • ReflectionRegistry now takes a ProxyRegistry to register proxy classes.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Apr 23, 2024
@graalvmbot graalvmbot force-pushed the lottet/GR-47365-proxy-mre branch 2 times, most recently from 8dbab50 to fe77ae7 Compare April 29, 2024 15:04
@graalvmbot graalvmbot closed this May 22, 2024
@graalvmbot graalvmbot deleted the lottet/GR-47365-proxy-mre branch May 22, 2024 07:52
@graalvmbot graalvmbot merged commit 82260b6 into master May 22, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants