[
https://issues.apache.org/jira/browse/OAK-7746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16610243#comment-16610243
]
Alex Deparvu commented on OAK-7746:
-----------------------------------
[~rombert], [~mreutegg] would like to hear your thoughts on the patch.
> Refactor TypePredicate location and usage
> -----------------------------------------
>
> Key: OAK-7746
> URL: https://issues.apache.org/jira/browse/OAK-7746
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: core, store-spi
> Reporter: Alex Deparvu
> Assignee: Alex Deparvu
> Priority: Major
> Labels: modularization
>
> In the context of modularization I would like to move the TypePredicate class
> to a dedicated package in store-spi (due to dependency on NodeState class).
> See [0] for some more reasoning.
> Proposed patch [1].
> I'm concerned about the Predicate (guava) and Predicate (jdk) usage. I think
> in a later guava release they will have Predicate extend the jdk version and
> advise people to start using the latter, so I think the patch is taking a
> step in a good direction, but would like to confirm this with others.
> Re. backwards compatibility, even if some packages are not exported (like
> observation) I did my best to deprecate and add newer alternatives. If this
> is not needed I can just drop the old methods.
> [0]
> https://issues.apache.org/jira/browse/OAK-7499?focusedCommentId=16486800&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16486800
> [1]
> https://github.com/apache/jackrabbit-oak/compare/trunk...stillalex:type-predicate
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
