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

Konrad Windszus commented on OAK-9840:
--------------------------------------

I updated the PR slightly to make the references towards MountInfoConfig 
dynamic 
(https://docs.osgi.org/specification/osgi.cmpn/8.0.0/service.component.html#service.component-reference.policy).
 It is threadsafe anyways and leads to more stable services. Also this makes 
the greedy option obsolete.

> Race condition in MountInfoProviderService
> ------------------------------------------
>
>                 Key: OAK-9840
>                 URL: https://issues.apache.org/jira/browse/OAK-9840
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: composite
>    Affects Versions: 1.44.0
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>
> The MountInfoConfig is bound reluctantly in 
> https://github.com/apache/jackrabbit-oak/blob/c424e1c179092594c365034c7b3f6eceb6cf2b42/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/MountInfoProviderService.java#L129,
>  apart from that the check in 
> https://github.com/apache/jackrabbit-oak/blob/c424e1c179092594c365034c7b3f6eceb6cf2b42/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/MountInfoProviderService.java#L140
>  is incorrect as it returns {{true}} if all found info configs are part of 
> the array {{expectedMounts}} but what is intended is rather that it only 
> returns true if all entries in the array are contained at least once in the 
> bound configs.



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

Reply via email to