[
https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15940670#comment-15940670
]
Stefan Seifert commented on SLING-6679:
---------------------------------------
i had a look as the patch from SLING-6694 - looks good to me.
one general question left: should we create a static instance of the factories
like JsonReaderFactory? this would avoid looking up the json provider and
intializing it again on every invocation.
according to
http://docs.oracle.com/javaee/7/api/javax/json/JsonReaderFactory.html it's
threadsafe.
same applies to the other factories.
on the other hand this may be a premature optimization, i've no metrics if it
really helps.
> Replace usage of org.apache.sling.commons.json.*
> ------------------------------------------------
>
> Key: SLING-6679
> URL: https://issues.apache.org/jira/browse/SLING-6679
> Project: Sling
> Issue Type: Improvement
> Reporter: Karl Pauls
> Assignee: Karl Pauls
>
> Following the deprecation of org.apache.sling.commons.json (SLING-6536) we
> need to replace its usage everywhere else (at least if we want to be able to
> release other modules that depend on it).
> This is the umbrella issue for getting this done. The idea is to create
> sub-issues with patches for individual components, review the patches, and
> when all are done: close this issue.
> General discussions and problems should go to this issue and specific ones on
> the sub-issue in question.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
