You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jackrabbit.apache.org by GitBox <gi...@apache.org> on 2021/08/19 15:35:12 UTC

[GitHub] [jackrabbit-oak] kwin commented on pull request #343: OAK-9541 javax.jcr.ItemExistsException during sysview import: add failing test

kwin commented on pull request #343:
URL: https://github.com/apache/jackrabbit-oak/pull/343#issuecomment-902015844


   According to https://github.com/apache/jackrabbit/blob/ed3124e5fe223dada33ce6ddf53bc666063c3f2f/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SerializationTest.java#L397 and https://docs.adobe.com/content/docs/en/spec/jsr170/javadocs/jcr-2.0/javax/jcr/Session.html#importXML(java.lang.String,%20java.io.InputStream,%20int) the `ItemExistsException` is actually correct. But that should IMHO be independent of the UUID behaviour (whether it conflicts or not) so the message "Node with the same UUID exists:" is definitely wrong here. So the question is rather: Under which circumstances should it not(!) throw an IEE (in case a node with the same name does already exist).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@jackrabbit.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org