[jira] [Comment Edited] (FELIX-6554) Exception while starting Felix Configurator

["Amit Mondal (Jira)" <ji...@apache.org>]

    [ 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)