[jira] [Commented] (JCRVLT-551) Set jcr:uuid to value from package when overwriting referenceable node

2021-08-17 Thread Tobias Bocanegra (Jira)


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

Tobias Bocanegra commented on JCRVLT-551:
-

I'm not sure anymore. afaicr, in jackrabbit, the referencers were stored with 
the referenceable node. i.e. double-linking.
i'm not sure if oak just does a search, when you try to get all referencers.

[~mreutegg] might know the details.

> Set jcr:uuid to value from package when overwriting referenceable node
> --
>
> Key: JCRVLT-551
> URL: https://issues.apache.org/jira/browse/JCRVLT-551
> Project: Jackrabbit FileVault
>  Issue Type: Improvement
>  Components: vlt
>Affects Versions: 3.5.0
>Reporter: Konrad Windszus
>Priority: Major
>
> Currently the code at 
> https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L809
>  does not try to adjust the {{jcr:uuid}} property in case a referenceable 
> node is imported from a package which already exists with a different 
> {{jcr:uuid}} in the repository.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (JCRVLT-551) Set jcr:uuid to value from package when overwriting referenceable node

2021-08-17 Thread Konrad Windszus (Jira)


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

Konrad Windszus commented on JCRVLT-551:


[~tripod] Any idea how JR2/Oak ensures referential integrity for the case that 
{{node-a}} existed with {{jcr:uuid=a}} before with some (non weak) references 
to it and is now replaced by {{node-a}} with {{jcr:uuid=b}} during the import?

> Set jcr:uuid to value from package when overwriting referenceable node
> --
>
> Key: JCRVLT-551
> URL: https://issues.apache.org/jira/browse/JCRVLT-551
> Project: Jackrabbit FileVault
>  Issue Type: Improvement
>  Components: vlt
>Affects Versions: 3.5.0
>Reporter: Konrad Windszus
>Priority: Major
>
> Currently the code at 
> https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L809
>  does not try to adjust the {{jcr:uuid}} property in case a referenceable 
> node is imported from a package which already exists with a different 
> {{jcr:uuid}} in the repository.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (JCRVLT-551) Set jcr:uuid to value from package when overwriting referenceable node

2021-08-17 Thread Konrad Windszus (Jira)


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

Konrad Windszus commented on JCRVLT-551:


Also using {{ImportUUIDBehavior.IMPORT_UUID_COLLISION_REMOVE_EXISTING}} is 
dangerous when the node did not exist before: 
https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L1022.
 I would rather recommend to always use 
https://docs.adobe.com/content/docs/en/spec/jcr/2.0/11_Import.html#11.8.4%20Throw%20on%20Identifier%20Collision.

> Set jcr:uuid to value from package when overwriting referenceable node
> --
>
> Key: JCRVLT-551
> URL: https://issues.apache.org/jira/browse/JCRVLT-551
> Project: Jackrabbit FileVault
>  Issue Type: Improvement
>  Components: vlt
>Affects Versions: 3.5.0
>Reporter: Konrad Windszus
>Priority: Major
>
> Currently the code at 
> https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L809
>  does not try to adjust the {{jcr:uuid}} property in case a referenceable 
> node is imported from a package which already exists with a different 
> {{jcr:uuid}} in the repository.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)