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

Merge policy cannot be defined for replicated maps with xml config #12223

Closed
blazember opened this issue Jan 30, 2018 · 2 comments
Closed

Merge policy cannot be defined for replicated maps with xml config #12223

blazember opened this issue Jan 30, 2018 · 2 comments

Comments

@blazember
Copy link
Contributor

@blazember blazember commented Jan 30, 2018

Currently, merge policy cannot be defined for replicated maps with xml configuration as merge-policy is missing in hazelcast-config.xsd for replicatedmap, and the same is true for xml based Spring configuration. The fix needs to be done in the following files:

  • hazelcast-config-3.10.xsd
  • hazelcast-spring-3.10.xsd
  • ConfigXmlGenerator.java
  • hazelcast-fullconfig.xml
  • fullConfig-applicationContext-hazelcast.xml
  • ConfigXmlGeneratorTest.java

Related #12120

@blazember blazember added this to the 3.10 milestone Jan 30, 2018
@Donnerbart
Copy link
Contributor

@Donnerbart Donnerbart commented Jan 31, 2018

The ReplicatedMap PR has not been merged yet. This should fix it: #12196

@blazember
Copy link
Contributor Author

@blazember blazember commented Feb 13, 2018

All listed places apart from ConfigXmlGeneratorTest have been covered by #12196. Only that one is in the scope.

blazember added a commit to blazember/hazelcast that referenced this issue Feb 14, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit to blazember/hazelcast that referenced this issue Feb 19, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit to blazember/hazelcast that referenced this issue Feb 20, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit to blazember/hazelcast that referenced this issue Feb 21, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit to blazember/hazelcast that referenced this issue Feb 21, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit to blazember/hazelcast that referenced this issue Feb 21, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes hazelcast#12223
blazember added a commit that referenced this issue Feb 22, 2018
Also verifying InMemoryFormat and AsyncFillup fields that were missing.

Fixes #12223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.