You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Julian Sedding (JIRA)" <ji...@apache.org> on 2016/11/22 10:41:58 UTC
[jira] [Closed] (SLING-6104) Improve handling to avoid Oak warning
[ https://issues.apache.org/jira/browse/SLING-6104?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Julian Sedding closed SLING-6104.
---------------------------------
> Improve handling to avoid Oak warning
> -------------------------------------
>
> Key: SLING-6104
> URL: https://issues.apache.org/jira/browse/SLING-6104
> Project: Sling
> Issue Type: Improvement
> Components: Installer, JCR
> Affects Versions: JCR Resource 2.8.0, JCR Installer 3.1.16
> Reporter: Jörg Hoh
> Assignee: Carsten Ziegeler
> Fix For: JCR Resource 2.8.2, JCR Installer 3.1.20, Pipes 0.0.10
>
>
> I often see messages like this in our logs (AEM 6.0):
> {noformat}
> WARN org.apache.jackrabbit.oak.jcr.session.ItemImpl Item#refresh invokes Session#refresh!
> {noformat}
> I found that in bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/JcrResourceUtil.java there is one occurrences of this pattern:
> {code}
> 444 } catch (RepositoryException re) {
> 445 // we ignore this as this folder might be created from a different task
> 446 node.refresh(false);
> 447 }
> {code}
> This should be changed to {code}node.getSession().refresh(false);{code} to avoid this warning. From my point of view the semantic does not change.
> I found the same pattern here as well:
> * installer/providers/jcr/src/main/java/org/apache/sling/installer/provider/jcr/impl/JcrUtil.java
> * contrib/extensions/sling-pipes/src/main/java/org/apache/sling/pipes/PathPipe.java
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)