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

Sunil Govindan commented on HDFS-14028:
---------------------------------------

Updating Fixed Version to 3.2.1 as this is fixed was not landed on branch-3.2.0 
which was closed for RC prep.

> HDFS OIV temporary dir deletes folder
> -------------------------------------
>
>                 Key: HDFS-14028
>                 URL: https://issues.apache.org/jira/browse/HDFS-14028
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs
>            Reporter: Adam Antal
>            Assignee: Adam Antal
>            Priority: Major
>             Fix For: 3.0.4, 3.1.2, 3.3.0, 3.2.1
>
>         Attachments: HDFS-14028.001.patch
>
>
> The Hadoop Offline Image Viewer tool has an undocumented 'feature' where it 
> will silently delete the directory passed in with the -t flag. This blew away 
> some important files when someone used a sensible, but ultimately poor choice 
> for this directory as the deletion isn't documented.
> For example, if someone were, as root do: 'hdfs oiv -i 
> fsimage_00000000000307052343 -p Delimited -t / -o image', bad things would 
> happen. This behavior should be documented and probably have a dialog or 
> throwing exception.
> There is a piece of code from PBImageTextWriter where a check can be added:
> {code:java}
> LevelDBMetadataMap(String baseDir) throws IOException {
>   File dbDir = new File(baseDir);
>   if (dbDir.exists()) {
>     FileUtils.deleteDirectory(dbDir);
>   }
>   ...
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to