shodaaan commented on PR #1805:
URL: https://github.com/apache/jackrabbit-oak/pull/1805#issuecomment-2444268556

   > > Regarding the explicit check, I don't think this is necessary
   > 
   > Fair enough.
   > 
   > Two unrelated additional comments:
   > 
   > * the DocumentNodeStore is not properly disposed. You can see this by 
running the [oak-run](https://issues.apache.org/jira/browse/OAK-run) command 
consecutively : the second start will have to wait 2min for the lease timeout 
of the first non-gracefully shut down run. We should ensure a graceful disposal.
   > * what might be useful is to add the base path where the garbage is 
generated to - such that the user immediately knows (and doesn't eg have to 
consult the source code)
   
   I added documentNodeStore.dispose() calls to CreateGarbageCommand. Thank you 
for the suggestion
   
   The root path where garbage is generated 
(/tmp/oak-run-generated-test-garbage) is specified in the USAGE string of 
CreateGarbageCommand, so users will know what the path is where garbage is 
generated.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to