[jira] [Commented] (OAK-3878) Avoid caching of NodeDocument while iterating in BlobReferenceIterator

["Chetan Mehrotra (JIRA)" <ji...@apache.org>]

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

Chetan Mehrotra commented on OAK-3878:
--------------------------------------

I was thinking to implement logic which bypasses DocumentStore similar to MongoVersionGCSupport for Blob GC which just expose the underlying cursor based iterator to logic above

[~catholicon] [~mreutegg] [~reschke] Thoughts? I can come up with patch if this approach looks ok

> Avoid caching of NodeDocument while iterating in BlobReferenceIterator
> ----------------------------------------------------------------------
>
>                 Key: OAK-3878
>                 URL: https://issues.apache.org/jira/browse/OAK-3878
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: documentmk
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>            Priority: Minor
>             Fix For: 1.8
>
>
> {{BlobReferenceIterator}} in DocumentMK makes use of {{DocumentStore}} API to query the NodeDocument. This would cause all those NodeDocuments to be added to cache in DocumentStore. Due to this when blob gc is running cache usage would not be that effective due to all the associated churn. 
> As these NodeDocument are only required for BlobGC logic and its not expected that this document would read again soon it would be better to skip caching of these documents within DocumentStore
> Similar requirement exist in VersionGC logic but there we use direct store based API which does not add such documents to the cache



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)