Alex Parvulescu resolved OAK-4450.
       Resolution: Fixed
         Assignee: Alex Parvulescu  (was: Michael Dürig)
    Fix Version/s:     (was: Segment Tar 0.0.18)
                   Segment Tar 0.0.16

fixed with http://svn.apache.org/viewvc?rev=1765293&view=rev
thanks [~mduerig] for the tight feedback loop.

> Properly split the FileStore into read-only and r/w variants 
> -------------------------------------------------------------
>                 Key: OAK-4450
>                 URL: https://issues.apache.org/jira/browse/OAK-4450
>             Project: Jackrabbit Oak
>          Issue Type: Technical task
>          Components: segment-tar
>            Reporter: Michael Dürig
>            Assignee: Alex Parvulescu
>              Labels: technical_debt
>             Fix For: Segment Tar 0.0.16
>         Attachments: OAK-4450-oak-run.patch
> The {{ReadOnlyFileStore}} class currently simply overrides the {{FileStore}} 
> class replacing all mutator methods with a trivial implementation. This 
> approach however leaks into its ancestor as the read only store needs to pass 
> a flag to the constructor of its super class so some fields can be 
> instantiated properly for the read only case. 
> We should clean this up to properly separate the read only and the r/w store. 
> Most likely we should factor the commonalities into a common, abstract base 
> class.

This message was sent by Atlassian JIRA

Reply via email to