Chetan Mehrotra commented on OAK-4924:

bq. Which means that we can't prefilter async index updates

Supporting that would require following change
# {{AsyncIndexUpdate#mergeWithConcurrencyCheck}} - It should pass a CommitInfo 
with {{CommitContext}} set. See {{MutableRoot}}
# You would need to explicitly include the ChangeCollector ValidatorProvider in 
the {{CompositeHook}}. 

This would then ensure that at least on cluster node where async index runs 
CommitInfo has ChangeSet populated

> avoid CommitInfo==null in contentChanged call in SegmentNodeStore.refresh
> -------------------------------------------------------------------------
>                 Key: OAK-4924
>                 URL: https://issues.apache.org/jira/browse/OAK-4924
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: segmentmk
>    Affects Versions: 1.5.12
>            Reporter: Stefan Egli
>            Assignee: Stefan Egli
> Currently 
> [SegmentNodeStore.refreshHead|https://github.com/apache/jackrabbit-oak/blob/d82f9b21d02a7eaf856f546e778118161d71b760/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeStore.java#L231]
>  calls {{contentChanged}} with null as the CommitInfo. This is problematic 
> for prefiltering (OAK-4796, OAK-4907 et al) as for that a ChangeSet of 
> changed items in a commit is stored in the CommitContext in the CommitInfo.
> Thus it would be useful to not send null but a CommitInfo that has the 
> equivalent meaning of null but is a new object for each call (so that the 
> ChangeSet can be set).
> [~mduerig], [~frm], [~alex.parvulescu] wdyt?

This message was sent by Atlassian JIRA

Reply via email to