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

ISPN-15494 Near cache with invalidation fails to start in Spring Boot #11727

Merged
merged 1 commit into from Jan 25, 2024

Conversation

jabolina
Copy link
Member

https://issues.redhat.com/browse/ISPN-15494

Updated for the invalidated near cache instances. We check if the MANAGER marshaller handles the server type and only then proceed with the usual flow. Either we register the listener and set the data format or the inverse.

If the marshaller is compatible with the server type, we exchange the type information when installing the listener. See, this might hide bugs that bite back during runtime! During runtime, when executing some key-based operation against the server, an exception is thrown because it is not possible to convert between types, failing later instead of during start-up. We also log a warning message.

Note that it would be a similar behavior to before we added #11337.

Internal caches, where the user won't interact directly through HR (resp), should be fine.

@jabolina jabolina added the 14.0.x Annotate a PR with this label if you want it to be backported to the 14.0.x branch label Jan 18, 2024
@jabolina jabolina added this to the 15.0.0.Dev07 milestone Jan 18, 2024
Copy link
Contributor

@karesti karesti left a comment

Choose a reason for hiding this comment

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

LGTM if ci is fine too

@jabolina
Copy link
Member Author

@karesti it seems OK. Some of the failures were handled in other PRs.

@jabolina
Copy link
Member Author

Let me rebase it.

Updated for the invalidated near cache instances. We check if the
MANAGER marshaller is capable of handling the server type, and only then
proceed with the usual flow. Either we register listener and set data
format, or the inverse.

If the marshaller is compatible with the server type, we exchange the
type information when installing the listener. See, this might hide bugs
that bite back during runtime! This might be found during runtime when
executing some key based operation against the server.

For internal caches, where the user won't interact directly through HR
(resp), this is fine. We also have a warning in any case.
@tristantarrant tristantarrant merged commit 5d461f0 into infinispan:main Jan 25, 2024
5 checks passed
@jabolina jabolina deleted the ISPN-15494 branch January 25, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14.0.x Annotate a PR with this label if you want it to be backported to the 14.0.x branch
Projects
None yet
3 participants