jackrabbit-oak icon indicating copy to clipboard operation
jackrabbit-oak copied to clipboard

Clarified a pitfall with --merge-paths

Open sbluhm opened this issue 6 years ago • 3 comments
trafficstars

Clarified a pitfall with a merge operation, should a node be renamed.

sbluhm avatar Nov 12 '19 13:11 sbluhm

@stefan-egli , any chance of merging this PR? It has been in my queue for a while.

sbluhm avatar Jul 23 '21 15:07 sbluhm

@sbluhm I believe this pitfall is only true for referenceable nodes. Other nodes don't have a UUID property, and thus are not subject to the same uniqueness constraint. Maybe you can verify and include this detail as well?

jsedding avatar Jul 26 '21 07:07 jsedding

@jsedding , thank you for your pointer. This indeed applies only to UUIDs. I just verified it. I have revised the statement and re-submitted.

sbluhm avatar Aug 21 '21 18:08 sbluhm

This PR is stale because it has been open 24 months with no activity. Remove stale label or comment or this will be closed in 30 days.

github-actions[bot] avatar Aug 22 '23 01:08 github-actions[bot]

@jsedding do you think we should incorporate this PR?

stefan-egli avatar Aug 29 '23 11:08 stefan-egli

@stefan-egli yes, I think this is helpful information and should be applied!

@sbluhm sorry this fell between the cracks!

jsedding avatar Aug 31 '23 07:08 jsedding