[ https://issues.apache.org/jira/browse/HDFS-14107?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16707259#comment-16707259 ]
BELUGA BEHR commented on HDFS-14107: ------------------------------------ Test is certainly related; I'll try to find time to come back to it. {code} org.apache.hadoop.fs.TestFileContextDeleteOnExit.testDeleteOnExit Took 0.51 sec. Error Message expected:<1> but was:<2> {code} > FileContext Delete on Exit Improvements > --------------------------------------- > > Key: HDFS-14107 > URL: https://issues.apache.org/jira/browse/HDFS-14107 > Project: Hadoop HDFS > Issue Type: Improvement > Components: fs > Affects Versions: 3.2.0 > Reporter: BELUGA BEHR > Assignee: BELUGA BEHR > Priority: Minor > Attachments: HADOOP-14107.2.patch, HDFS-14107.1.patch > > > {code:java|FileContext.java} > synchronized (DELETE_ON_EXIT) { > Set<Entry<FileContext, Set<Path>>> set = DELETE_ON_EXIT.entrySet(); > for (Entry<FileContext, Set<Path>> entry : set) { > FileContext fc = entry.getKey(); > Set<Path> paths = entry.getValue(); > for (Path path : paths) { > try { > fc.delete(path, true); > } catch (IOException e) { > LOG.warn("Ignoring failure to deleteOnExit for path " + path); > } > } > } > DELETE_ON_EXIT.clear(); > {code} > # Include the {{IOException}} in the logging so that admins can know why the > file was not deleted > # Do not bother clearing out the data structure. This code is only called if > the JVM is going down. Better to spend the time allowing another shutdown > hook to run than to spend time cleaning this thing up. > # Use Guava {{MultiMap}} for readability > # Paths are currently stored in a {{TreeSet}}. This set implementation > orders the files by names. It does not seem worth much to order the files. > Use a faster {{HashSet}}. -- 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