[
https://issues.apache.org/jira/browse/JCRVLT-745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17817442#comment-17817442
]
Julian Reschke commented on JCRVLT-745:
---------------------------------------
Q: why is "/" the first choice, not "/tmp"?
https://github.com/apache/jackrabbit-filevault/blame/81fa88f1d7af07e77871e2c0bb03515107f18d74/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/NodeStash.java#L60-L66
Also: was it considered to do the first part of the stashing operation just to
move the entired node (plus descendants)? That would essentially half the
number of required move operations...
> Stashing: naming and folder location
> ------------------------------------
>
> Key: JCRVLT-745
> URL: https://issues.apache.org/jira/browse/JCRVLT-745
> Project: Jackrabbit FileVault
> Issue Type: Improvement
> Components: vlt
> Reporter: Julian Reschke
> Assignee: Julian Reschke
> Priority: Major
>
> We have seen cases where node stashing failed due to RuntimeExceptions (OOM
> or MongoDB issues in Oak). In these cases, the tmp folder is not cleaned up.
> If the operation is retried many times, there'll be many of these.
> Suggestion:
> 1. Do not use the root as default location,
> 2. Introduce an intermediary node that makes it clear that this is temp space
> created by FileVault.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
