Skip to content

Conversation

@chrisdennis
Copy link
Member

@chrisdennis chrisdennis commented Aug 20, 2025

We move back here to a non-lazy approach to building the map of 107 wrappers. This means we generate all the wrappers up front, and then we generate new wrappers by listening for added caches, and remove when we see a cache removed. This should mean that direct additions/removals to the Ehcache manager will be immediately reflected in the JSR-107 manager.

Copy link
Member

@jhouserizer jhouserizer left a comment

Choose a reason for hiding this comment

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

Pretty hairy changes, but I don't see anything amiss.

@chrisdennis chrisdennis marked this pull request as ready for review August 22, 2025 15:21
@chrisdennis chrisdennis requested review from prasantakumar82 and removed request for tmesic99 August 27, 2025 17:02
@chrisdennis
Copy link
Member Author

Local testing confirms this fixes the issue in the spring-boot test, and triggers no other failures there.

Copy link
Contributor

@prasantakumar82 prasantakumar82 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@prasantakumar82 prasantakumar82 merged commit f5d1a7e into ehcache:master Aug 29, 2025
7 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