jackrabbit-oak-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Francesco Mari (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OAK-6001) FileStore.GarbageCollector#collectBulkReferences performs unnecessary operations
Date Wed, 29 Mar 2017 12:47:41 GMT

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

Francesco Mari commented on OAK-6001:

[~mduerig], can you double-check this?

> 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
> {{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