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

ConcurrentModificationException under load #402

Open
ab0159w opened this issue Dec 8, 2022 · 1 comment
Open

ConcurrentModificationException under load #402

ab0159w opened this issue Dec 8, 2022 · 1 comment

Comments

@ab0159w
Copy link

ab0159w commented Dec 8, 2022

Hi,

We are seeing this error under load.

java.util.ConcurrentModificationException: null
	at java.base/java.util.TreeMap$PrivateEntryIterator.nextEntry(Unknown Source)
	at java.base/java.util.TreeMap$KeyIterator.next(Unknown Source)
	at org.jeasy.rules.core.RuleProxy.appendActionMethodsNames(RuleProxy.java:353)
	at org.jeasy.rules.core.RuleProxy.getRuleDescription(RuleProxy.java:334)
	at org.jeasy.rules.core.RuleProxy.invoke(RuleProxy.java:101)
	at com.sun.proxy.$Proxy218.getDescription(Unknown Source)
	at org.jeasy.rules.core.DefaultRulesEngine.log(DefaultRulesEngine.java:146)
	at org.jeasy.rules.core.DefaultRulesEngine.doFire(DefaultRulesEngine.java:80)
	at org.jeasy.rules.core.DefaultRulesEngine.fire(DefaultRulesEngine.java:70)

In RuleProxy line 279

    private Set<ActionMethodOrderBean> getActionMethodBeans() {
        if (this.actionMethods == null) {
            this.actionMethods = new TreeSet<>();

Consider using ConcurrentSkipListSet.

@dvgaba
Copy link

dvgaba commented Dec 13, 2022

Could you please share a sample test/git repo to re-produce this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants