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

Neil Joshi edited comment on HDDS-8162 at 3/17/23 8:32 PM:
-----------------------------------------------------------

HDDS-7383 defined its tmp directory in StorageVolume, while HDDS-6449 defined 
the tmp directory in HddsVolume.  I made a comment for the discrepancy in the 
PR review, unfortunately missed the follow-up.  It needs to me updated with 
HDDS-6449.  see,

[https://github.com/apache/ozone/pull/3874/files/40bebd8f7e1a92f3b8b971aba253660bb4f19f44#r1004131893]
 

cc [~Symious] , [~sammichen] 


was (Author: neiljoshi):
HDDS-7383 defined its tmp directory in StorageVolume, while HDDS-6449 defined 
the tmp directory in HddsVolume.  I made of comment for the discrepancy in the 
PR review, unfortunately missed the follow-up.  It needs to me updated with 
HDDS-6449.  see,

https://github.com/apache/ozone/pull/3874/files/40bebd8f7e1a92f3b8b971aba253660bb4f19f44#r1004131893
 

cc [~Symious] , [~sammichen] 

> Reconcile tmp directories for different features
> ------------------------------------------------
>
>                 Key: HDDS-8162
>                 URL: https://issues.apache.org/jira/browse/HDDS-8162
>             Project: Apache Ozone
>          Issue Type: Sub-task
>            Reporter: Ethan Rose
>            Priority: Major
>
> HDDS-7083 put its temp directory in /hdds/tmp. HDDS-6449 in 
> /hdds/<cluster-id>/tmp. IMO the second location is better since it provides 
> better isolation from the rest of the filesystem at the cluster level. We 
> should have all operations that use a tmp directory use the same parent to 
> put their subdirectories under, for example 
> /hdds/<cluster-id>/container-delete and /hdds/<cluster-id>/tmp/container-copy.
>  
> HDDS-7383 defined its tmp directory in StorageVolume, while HDDS-6449 defined 
> the tmp directory in HddsVolume.  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to