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
Copy Nearcacheconfig when it needs to be changed #18348
Conversation
This reverts commit 8df8a6e.
hazelcast/src/test/java/com/hazelcast/config/NearCacheConfigAccessorTest.java
Outdated
Show resolved
Hide resolved
hazelcast/src/test/java/com/hazelcast/config/NearCacheConfigAccessorTest.java
Show resolved
Hide resolved
* Used by the {@link NearCacheConfigAccessor} to | ||
* initialize the proper default value for on-heap maps. | ||
*/ | ||
boolean sizeConfigured; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if sizeConfigured
was set to true
, after deserialization what keeps it still true
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have reverted the EvictionConfig behaviour. So it seems that setting sizeConfigured on deserialization was not necessary. For the sake of keeping it backward compatible, I will not touch this. Lets consider if that is necessary as a separate fix if needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, i see this is an existing issue, not introduced in this PR. Created an issue for it: #18354
hazelcast/src/main/java/com/hazelcast/config/NearCacheConfigAccessor.java
Outdated
Show resolved
Hide resolved
a7747a2
to
e5051d2
Compare
Copies NearcacheConfig and EvictionConfig when they needs to be changed by NearcacheConfigAccessor so that the object on ClientConfig/Config does not change when used. fixes hazelcast#17952
This reverts commit 8df8a6e.
Copies NearcacheConfig and EvictionConfig when they needs
to be changed by NearcacheConfigAccessor so that the object
on ClientConfig/Config does not change when used.
fixes #17952
fixes https://github.com/hazelcast/hazelcast-enterprise/issues/3966