[
https://issues.apache.org/jira/browse/OAK-800?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13645347#comment-13645347
]
Tommaso Teofili commented on OAK-800:
-------------------------------------
bq. Good point. We'll need to make sure that the type compilation done by
RegistrationEditor is automatically applied when upgrading an existing
repository (mostly Jackrabbit ones, not so important yet for earlier Oak
versions).
I agree it's more important for JR installations
bq. Alternatively we could make the TypeEditor automatically compile the type
information in memory if it's not yet available in content.
I'm not yet sure about what's the best alternative here:
* the former (re applying _RegistrationEditor_'s type compilation), in my
opinion, looks better at a first glance even if I'm not sure modifying node
types information on the repository would sound nice to users / consumers of
the repository
* the latter (_TypeEditor_'s in memory type compilation) would probably be more
costly but less intrusive even if once you upgrade you're perhaps expecting
some changes to happen.
Probably that's also part of a broader discussion on migration from JR to Oak.
> EffectiveType#getTypeNames may throw NPE
> ----------------------------------------
>
> Key: OAK-800
> URL: https://issues.apache.org/jira/browse/OAK-800
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: core
> Reporter: Tommaso Teofili
> Assignee: Tommaso Teofili
> Priority: Minor
> Fix For: 0.7
>
>
> _EffectiveType#getTypeNames_ may throw NPE if any of the types doesn't have
> _oak:supertypes_ property defined.
> At line 200:
> {code}
> Iterables.addAll(names,
> type.getProperty("oak:supertypes").getValue(NAMES));
> {code}
> where _type.getProperty("oak:supertypes")_ may be _null_ (as per
> _@CheckForNull_ annotation on _NodeState#getProperty(String name)_).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
