[ https://issues.apache.org/jira/browse/HBASE-6356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13490628#comment-13490628 ]
nkeywal commented on HBASE-6356: -------------------------------- Hi Gustavo, Yes, it would work. It's the simple option. But it's clean. I would add a message in the log, especially mentioning that we're ignoring the error and returning false. Do you want to submit a patch? Thanks, > printStackTrace in FSUtils > -------------------------- > > Key: HBASE-6356 > URL: https://issues.apache.org/jira/browse/HBASE-6356 > Project: HBase > Issue Type: Bug > Components: Client, master, regionserver > Affects Versions: 0.96.0 > Reporter: nkeywal > Priority: Trivial > Labels: noob > > This is bad... > {noformat} > public boolean accept(Path p) { > boolean isValid = false; > try { > if (HConstants.HBASE_NON_USER_TABLE_DIRS.contains(p.toString())) { > isValid = false; > } else { > isValid = this.fs.getFileStatus(p).isDir(); > } > } catch (IOException e) { > e.printStackTrace(); <================ > } > return isValid; > } > } > {noformat} -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira