jackrabbit-oak-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Dürig (JIRA) <j...@apache.org>
Subject [jira] [Commented] (OAK-6001) FileStore.GarbageCollector#collectBulkReferences performs unnecessary operations
Date Thu, 30 Mar 2017 15:13:41 GMT

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

Michael Dürig commented on OAK-6001:

But the call to {{TarWriter#collectReferences}} was necessary at the time I fixed OAK-4700
(https://github.com/apache/jackrabbit-oak/commit/9bee082). Commenting out just that call at
that revision causes {{CompactionAndCleanupIT#concurrentCleanup}} to fail frequently. So my
question is what changes to other parts in the code made {{TarWriter#collectReferences}} superfluous
later on? I will probably have to {{git bisect}} this later today...

> FileStore.GarbageCollector#collectBulkReferences performs unnecessary operations
> --------------------------------------------------------------------------------
>                 Key: OAK-6001
>                 URL: https://issues.apache.org/jira/browse/OAK-6001
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: segment-tar
>            Reporter: Francesco Mari
>            Assignee: Francesco Mari
>             Fix For: 1.8
> {{FileStore.GarbageCollector#collectBulkReferences}} looks into the currently active
{{TarWriter}} for references. At the time this method is invoked, the currently active {{TarWriter}}
is always empty.
> Slightly before inspecting the {{TarWriter}}, {{FileStore#newWriter}} is called to flush
the content of the current {{TarWriter}} and to create a fresh one. For this reason, inspecting
the {{TarWriter}} in {{FileStore.GarbageCollector#collectBulkReferences}} is unnecessary -
the graph of the {{TarWriter}} is always empty. Those lines of code can be safely removed.

This message was sent by Atlassian JIRA

View raw message