jackrabbit-oak-issues mailing list archives

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

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

Alex Parvulescu commented on OAK-6001:
--------------------------------------

bq. Those lines of code can be safely removed.
yep, looks like it. indeed, the flush call is what renders the call useless.

> 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
(v6.3.15#6346)

Mime
View raw message