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
Introduced system property to provide backward #13501
Introduced system property to provide backward #13501
Conversation
1a3f5d5
to
55b6b3e
Compare
@@ -72,6 +73,9 @@ | |||
@SuppressWarnings("WeakerAccess") | |||
public class MapContainer { | |||
|
|||
public static final HazelcastProperty LOAD_ALL_PUBLISHES_ADD_EVENT |
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.
When should this property be enabled? How does an end user know this property needs to be set?
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.
Intention is to provide a soft pass. Because firing LOADED events upon loadAll call is not a backward compatible change. This system property can be removed in 4.0. Doc update is needed to inform user.
@@ -140,11 +146,16 @@ public MapContainer(final String name, final Config config, final MapServiceCont | |||
} else { | |||
this.globalIndexes = null; | |||
} | |||
this.addEventPublishingEnabled = nodeEngine.getProperties().getBoolean(LOAD_ALL_PUBLISHES_ADD_EVENT); |
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.
Can we switch to nodeEngine.getProperties().getBoolean(LOAD_ALL_PUBLISHES_ADD_EVENT)
and have it in group properties?
@@ -72,6 +73,9 @@ | |||
@SuppressWarnings("WeakerAccess") | |||
public class MapContainer { | |||
|
|||
public static final HazelcastProperty LOAD_ALL_PUBLISHES_ADD_EVENT |
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.
Can you add some javadoc for the property?
d353b2e
to
46431c1
Compare
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.
Consider just moving the property to GroupProperty
with the rest
fab543e
to
f0d6cff
Compare
compatible event firing behavior for map#loadAll. When compatibility mode is on loadAll will fire ADD events, when off, it will fire LOADED event. By default LOADED events will be fired.
f0d6cff
to
5f5dec6
Compare
compatible event firing behavior for map#loadAll.
When compatibility mode is on loadAll will fire ADD events,
when off, it will fire LOADED event. By default LOADED events
will be fired.