Skip to content

[NAE-2449] ActionDelegate memory leak causes OOM#452

Draft
renczesstefan wants to merge 3 commits into
release/7.0.0-rev10from
NAE-2449
Draft

[NAE-2449] ActionDelegate memory leak causes OOM#452
renczesstefan wants to merge 3 commits into
release/7.0.0-rev10from
NAE-2449

Conversation

@renczesstefan

@renczesstefan renczesstefan commented Jun 11, 2026

Copy link
Copy Markdown
Member

Description

Refactor action handling logic and improve memory management in FieldActionsRunner and related classes.

Fixes NAE-2449

Dependencies

No new dependencies were introduced

Third party dependencies

No new dependencies were introduced

Blocking Pull requests

There are no dependencies on other PR

How Has Been This Tested?

This was tested manually and with unit tests.

Test Configuration

Name Tested on
OS macOS Tahoe 26.3
Runtime Java 21
Dependency Manager Maven 3.9.9n
Framework version Spring Boot 3.4.4
Run parameters
Other configuration

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @machacjozef
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

Refactor action handling logic and improve memory management in FieldActionsRunner and related classes.
@renczesstefan renczesstefan self-assigned this Jun 11, 2026
@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 32b88bcb-208e-4b70-a22b-ccff28dc89aa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Introduce FunctionExpando as a lightweight proxy for scoped global functions,
delegating property and method access to the parent ActionDelegate while
rehydrating cached function closures only when invoked.

Replace eager global function rehydration in FieldActionsRunner with lazy
FunctionExpando-based binding to reduce heap usage during action preparation.
…cher to support dynamic method invocation and improve closure signature matching.
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