[
https://issues.apache.org/jira/browse/OAK-7710?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16608995#comment-16608995
]
Tomek Rękawek edited comment on OAK-7710 at 9/10/18 11:08 AM:
--------------------------------------------------------------
[~mreutegg] I see. So, maybe we should remove the
{{dispatcher.contentChanged()}} invocation from the {{merge()}} method at all?
All the events should be announced from the MountedNodeStoreObserver registered
to observe the global node store: [^OAK-7710-3.patch] WDYT?
was (Author: tomek.rekawek):
[~mreutegg] I see. So, maybe we should remove the
{{dispatcher.contentChanged()}} invocation from the {{merge()}} method at all?
All the events should be announced from the MountedNodeStoreObserver registered
to observe the global node store. WDYT?
> CompositeNodeStore does not dispatch external events to observers
> -----------------------------------------------------------------
>
> Key: OAK-7710
> URL: https://issues.apache.org/jira/browse/OAK-7710
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: composite
> Reporter: Vikas Saurabh
> Assignee: Tomek Rękawek
> Priority: Major
> Fix For: 1.10
>
> Attachments: OAK-7710-2.patch, OAK-7710-3.patch, OAK-7710.patch,
> OAK-7710.test.patch
>
>
> Currently {{CompositeNodeStore}} only ever dispatches changes from inside its
> {{merge}} method. This then loses external events that could be read in
> background read of some underlying {{DocumentNodeStore}}.
> [^OAK-7710.test.patch] has an ignored test representing the scenario.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
