[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14219164#comment-14219164 ] metatech commented on FELIX-3807: - I tried this patch in an attempt to solve FELIX-3067,

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13567886#comment-13567886 ] Richard S. Hall commented on FELIX-3807: I needed to commit one additional patch

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531149#comment-13531149 ] Guillaume Nodet commented on FELIX-3807: The bundle lock can be acquired before

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531166#comment-13531166 ] Richard S. Hall commented on FELIX-3807: Ok, I think I understand. The refresh

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531179#comment-13531179 ] Guillaume Nodet commented on FELIX-3807: Here is a proposed patch: {code} diff

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531184#comment-13531184 ] Richard S. Hall commented on FELIX-3807: Still, you don't need to grab all locks

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531187#comment-13531187 ] Richard S. Hall commented on FELIX-3807: Unlocking isn't necessary as a separate

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531189#comment-13531189 ] Guillaume Nodet commented on FELIX-3807: That's true, I'll refactor the patch to

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531204#comment-13531204 ] Guillaume Nodet commented on FELIX-3807: Revised patch {code} diff --git

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531207#comment-13531207 ] Guillaume Nodet commented on FELIX-3807: Which leads to the following code: {code}

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13531218#comment-13531218 ] Richard S. Hall commented on FELIX-3807: Looks good to me. Go ahead and commit it

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13526701#comment-13526701 ] Richard S. Hall commented on FELIX-3807: I'm not sure why this would be necessary.

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13526737#comment-13526737 ] Guillaume Nodet commented on FELIX-3807: Maybe the bundle lock was acquired before

[jira] [Commented] (FELIX-3807) Refreshing bundles should first grab all the bundle locks to avoid concurrent modifications of those bundles

[ https://issues.apache.org/jira/browse/FELIX-3807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13526742#comment-13526742 ] Richard S. Hall commented on FELIX-3807: Yes, current bundle lock holders can