[jira] [Commented] (JCR-2961) Improve logging of Session.save() to trace back root cause of externally modified nodes

[Thomas Mueller (JIRA)]

    [ 
https://issues.apache.org/jira/browse/JCR-2961?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13030647#comment-13030647
 ] 

Thomas Mueller commented on JCR-2961:
-------------------------------------

Most likely you don't see the NPE because it is ignored (now) when using the 
default log settings. To see it in stdout, use 
src/test/resource/lockback-test.xml:


<?xml version="1.0" encoding="UTF-8"?>
<configuration>

  <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
    <layout class="ch.qos.logback.classic.PatternLayout">
      <Pattern>%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - 
%msg%n</Pattern>
    </layout>
  </appender>

  <appender name="file" class="ch.qos.logback.core.FileAppender">
    <file>target/jcr.log</file>
    <encoder>
      <pattern>%date{HH:mm:ss.SSS} %-5level %-40([%thread] %F:%L) 
%msg%n</pattern>
    </encoder>
  </appender>

  <appender name="sift" class="ch.qos.logback.classic.sift.SiftingAppender">
    <discriminator>
      <Key>testclass</Key>
      <DefaultValue>junit</DefaultValue>
    </discriminator>
    <sift>
      <appender name="FILE-${testname}" 
class="ch.qos.logback.core.FileAppender">
        <File>target/surefire-reports/${testclass}.log</File>
        <layout class="ch.qos.logback.classic.PatternLayout">
          <Pattern>%date{HH:mm:ss.SSS} %-5level %-40([%thread] %F:%L) 
%msg%n</Pattern>
        </layout>
      </appender>
    </sift>
  </appender>

  <logger name="org.apache.jackrabbit.test" level="DEBUG"/>

  <root level="INFO">
    <appender-ref ref="file"/>
    <appender-ref ref="sift"/>
  </root>

  <logger name="org.apache.jackrabbit.core.session.SessionSaveOperation" 
level="DEBUG">
    <appender-ref ref="STDOUT"/>
  </logger>

</configuration>


> Improve logging of Session.save() to trace back root cause of externally 
> modified nodes
> ---------------------------------------------------------------------------------------
>
>                 Key: JCR-2961
>                 URL: https://issues.apache.org/jira/browse/JCR-2961
>             Project: Jackrabbit Content Repository
>          Issue Type: New Feature
>            Reporter: Thomas Mueller
>            Assignee: Thomas Mueller
>            Priority: Minor
>             Fix For: 2.3.0
>
>
> Currently it's very difficult to find the root cause of error like: 
> javax.jcr.InvalidItemStateException: <UUID> has been modified externally.
> To better trace back such issues, it would be nice to add DEBUG logging for 
> the Session.save() call.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira