[ https://issues.apache.org/jira/browse/FELIX-5443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15769149#comment-15769149 ]
Carsten Ziegeler commented on FELIX-5443: ----------------------------------------- :) I'm not sure if locking all PID is really increasing complexity. Looking at your patch, you do a locking on the factory (pid) which is basically the same as I suggest. The difference being that your patch does it only for factory configurations > Frequent Changes cause UpdateThread to ConcurrentModificationException > ---------------------------------------------------------------------- > > Key: FELIX-5443 > URL: https://issues.apache.org/jira/browse/FELIX-5443 > Project: Felix > Issue Type: Bug > Components: Configuration Admin > Affects Versions: configadmin-1.8.8 > Reporter: Fabian Lange > Fix For: configadmin-1.9.0 > > > 2016-11-30T10:20:15.614+0000 | ERROR | 836-1bc8-4066-963b-9b9cbccbd9d0) | > configadmin | org.apache.felix.configadmin - 1.8.8 | Unexpected problem > executing task java.util.ConcurrentModificationException at > java.util.HashMap$HashIterator.nextNode(HashMap.java:1437)[:1.8.0_101] at > java.util.HashMap$KeyIterator.next(HashMap.java:1461)[:1.8.0_101] at > java.util.AbstractCollection.toArray(AbstractCollection.java:196)[:1.8.0_101] > at > org.apache.felix.cm.impl.Factory.store(Factory.java:126)[4:org.apache.felix.configadmin:1.8.8] > at > org.apache.felix.cm.impl.ConfigurationManager$DeleteConfiguration.run(ConfigurationManager.java:1851)[4:org.apache.felix.configadmin:1.8.8] > at > org.apache.felix.cm.impl.UpdateThread.run0(UpdateThread.java:143)[4:org.apache.felix.configadmin:1.8.8] > at > org.apache.felix.cm.impl.UpdateThread.run(UpdateThread.java:110)[4:org.apache.felix.configadmin:1.8.8] > at java.lang.Thread.run(Thread.java:745)[:1.8.0_101] -- This message was sent by Atlassian JIRA (v6.3.4#6332)