[ https://issues.apache.org/jira/browse/HADOOP-13765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614071#comment-15614071 ]
Yuanbo Liu commented on HADOOP-13765: ------------------------------------- LGTM, [~byh0831] Thanks for filing this jira. [~ste...@apache.org] I also checked the behavior of {{FTPFileSystem}}, it throws a runtime exception if there is a running error. I'm not sure which behavior is more reasonable, looking forward to your thoughts. > Return HomeDirectory if possible in SFTPFileSystem > -------------------------------------------------- > > Key: HADOOP-13765 > URL: https://issues.apache.org/jira/browse/HADOOP-13765 > Project: Hadoop Common > Issue Type: Bug > Components: fs > Reporter: Yuhao Bi > Attachments: HADOOP-13765.001.patch > > > In SFTPFileSystem#getHomeDirectory(), we disconnect the ChannelSftp in > finally block. > If we get the homeDir Path successfully but got an IOE in the finally block > we will return the null result. > Maybe we can simply ignore this IOE and just return the result we have got. > Related codes are shown below. > {code:title=SFTPFileSystem.java|borderStyle=solid} > public Path getHomeDirectory() { > ChannelSftp channel = null; > try { > channel = connect(); > Path homeDir = new Path(channel.pwd()); > return homeDir; > } catch (Exception ioe) { > return null; > } finally { > try { > disconnect(channel); > } catch (IOException ioe) { > //Maybe we can just ignore this IOE and do not return null here. > return null; > } > } > } > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org