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

WFLY-18036 Marshalling optimizations are not getting applied to @SessionScoped @Stateful EJBs #16859

Merged
merged 4 commits into from May 30, 2023

Conversation

pferraro
Copy link
Contributor

@pferraro pferraro requested a review from rhusar May 19, 2023 13:25
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label May 19, 2023
@pferraro
Copy link
Contributor Author

Rebasing so that we can test this along with recent fixes.

@darranl
Copy link
Contributor

darranl commented May 24, 2023

@rhusar looks like one more here asking for your review I may be able to bundle in if you are Ok with it?

Copy link
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good (also we have pretested this in a subset of QE tests) - cc @darranl

@darranl darranl added the ready-for-merge Only for use by those with merge permissions! label May 25, 2023
@darranl
Copy link
Contributor

darranl commented May 26, 2023

/retest

@darranl darranl mentioned this pull request May 30, 2023
@darranl darranl merged commit 461a97c into wildfly:main May 30, 2023
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
29.Alpha1 deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge Only for use by those with merge permissions!
Projects
None yet
3 participants