[ 
https://issues.apache.org/jira/browse/FELIX-6554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17577242#comment-17577242
 ] 

Amit Mondal edited comment on FELIX-6554 at 8/9/22 8:05 AM:
------------------------------------------------------------

[~cziegeler] 

Thanks a lot for your continuous support. I had a thorough testing locally 
where our management agent started installing the configurator and the 
exception occurred a few times. After delving deep into it, I strongly believe 
it is not due to concurrency issues where multiple threads accessing the 
internal data, it is due to the same thread trying to modify the map(s) while 
iterating it (as suggested by [~laeubi]). I think your fix should solve the 
problem.

What do you suggest then? Should I close the PR?


was (Author: amitmondal):
[~cziegeler] 

Thanks a lot for your continuous support. I had a thorough testing locally 
where our management agent started installing the configurator and the 
exception occurred a few times. After delving deep into it, I strongly believe 
it is not due to concurrency issues where multiple threads accessing the 
internal data, it is due to the same thread trying to modify the map(s) while 
iterating it (as suggested by [~laeubi]). I think your fix should solve the 
problem.

What do you suggest then? Should I close the MR?

> Exception while starting Felix Configurator
> -------------------------------------------
>
>                 Key: FELIX-6554
>                 URL: https://issues.apache.org/jira/browse/FELIX-6554
>             Project: Felix
>          Issue Type: Bug
>          Components: Configurator
>    Affects Versions: configurator-1.0.14
>            Reporter: Amit Mondal
>            Priority: Minor
>              Labels: pull-request-available
>
> {code:java}
> java.util.ConcurrentModificationException
> at java.util.HashMap$HashIterator.nextNode(HashMap.java:1469)
> at java.util.HashMap$KeyIterator.next(HashMap.java:1493)
> at 
> org.apache.felix.configurator.impl.Configurator.start(Configurator.java:291)
> at 
> org.apache.felix.configurator.impl.ServicesListener.notifyChange(ServicesListener.java:117)
> at 
> org.apache.felix.configurator.impl.ServicesListener$1.addingService(ServicesListener.java:73)
> at 
> org.apache.felix.configurator.impl.ServicesListener$1.addingService(ServicesListener.java:65)
> at 
> org.osgi.util.tracker.ServiceTracker$Tracked.customizerAdding(ServiceTracker.java:944)
> at 
> org.osgi.util.tracker.ServiceTracker$Tracked.customizerAdding(ServiceTracker.java:872)
> at org.osgi.util.tracker.AbstractTracked.trackAdding(AbstractTracked.java:256)
> at 
> org.osgi.util.tracker.AbstractTracked.trackInitial(AbstractTracked.java:183)
> at org.osgi.util.tracker.ServiceTracker.open(ServiceTracker.java:322)
> at org.osgi.util.tracker.ServiceTracker.open(ServiceTracker.java:265)
> at 
> org.apache.felix.configurator.impl.ServicesListener.<init>(ServicesListener.java:93)
> at org.apache.felix.configurator.impl.Activator.start(Activator.java:36)
> at 
> org.apache.felix.framework.util.SecureAction.startActivator(SecureAction.java:849)
> at org.apache.felix.framework.Felix.activateBundle(Felix.java:2429){code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to