[ 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