[
https://issues.apache.org/jira/browse/HDFS-15886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17300260#comment-17300260
]
Max Xie commented on HDFS-15886:
---------------------------------
Thank you for tomscut and Stephen review.
It's a great idea to use a mixture of paths and files to get protected dirs
and I will try to code it.
As for reload the config contents, maybe we can add a new protocol like
refreshProtectedDirectories (similar to refreshNodes command) instead of
reconfigging fs.protected.directories by calling
Namenode.reconfProtectedDirectories.
And I have updated the patch. Please take a look.
> Add a way to get protected dirs from a special configuration file
> -----------------------------------------------------------------
>
> Key: HDFS-15886
> URL: https://issues.apache.org/jira/browse/HDFS-15886
> Project: Hadoop HDFS
> Issue Type: New Feature
> Components: namenode
> Affects Versions: 3.4.0
> Reporter: Max Xie
> Assignee: Max Xie
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> We used protected dirs to ensure that important data directories cannot be
> deleted by mistake. But protected dirs can only be configured in
> hdfs-site.xml.
> For ease of management, we add a way to get the list of protected dirs from
> a special configuration file.
> How to use.
> 1. set the config in hdfs-site.xml
> ```
> <property>
> <name>fs.protected.directories</name>
>
> <value>/hdfs/path/1,/hdfs/path/2,[file:///path/to/protected.dirs.config]</value>
> </property>
> ```
> 2. add some protected dirs to the config file
> ([file:///path/to/protected.dirs.config])
> ```
> /hdfs/path/4
> /hdfs/path/5
> ```
> 3. use command to refresh fs.protected.directories instead of
> FSDirectory.setProtectedDirectories(..)
> ```
> hdfs dfsadmin -refreshProtectedDirectories
> ```
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
