[ https://issues.apache.org/jira/browse/OAK-5046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15632966#comment-15632966 ]
Andrei Dulceanu commented on OAK-5046: -------------------------------------- bq Also, How does the current code deal with disabling the estimation phase? Let's say I want to skip estimation, how do I do that? One way to do it would be:{{SegmentGCOptions.defaultGCOptions().disableEstimation()}}. If you are OK with it and everything else I will create a new patch including the two other suggestions from above. > Remove the old estimation OSGi setting (compaction.gainThreshold) > ----------------------------------------------------------------- > > Key: OAK-5046 > URL: https://issues.apache.org/jira/browse/OAK-5046 > Project: Jackrabbit Oak > Issue Type: Improvement > Components: segment-tar > Affects Versions: 1.5.12 > Reporter: Andrei Dulceanu > Assignee: Andrei Dulceanu > Priority: Minor > Labels: osgi-config > Fix For: 1.5.13 > > Attachments: OAK-5046-01.patch, OAK-5046-02.patch > > > Currently, there are two implementations for finding out the gain in > repository size after running compaction: the old one, > {{CompactionGainEstimate}} and the new one, {{SizeDeltaGcEstimation}}. > Similarly, there are also two configurations for customising them, in > {{SegmentNodeStoreService}}, {{compaction.gainThreshold}} and > {{compaction.sizeDeltaEstimation}}. > At the moment both of them are exposed as OSGi configurations, but only the > new one should be exposed (e.g. {{compaction.sizeDeltaEstimation}}). > It must be evaluated whether it makes sense to keep the logic associated with > the old implementation. -- This message was sent by Atlassian JIRA (v6.3.4#6332)