You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Gustavo Anatoly (JIRA)" <ji...@apache.org> on 2012/11/05 00:42:12 UTC
[jira] [Commented] (HBASE-6356) printStackTrace in FSUtils
[ https://issues.apache.org/jira/browse/HBASE-6356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13490330#comment-13490330 ]
Gustavo Anatoly commented on HBASE-6356:
----------------------------------------
Hi, nkeywal.
I read about your comment and I make a simple update like this:
{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) {
LOG.warn(e);
}
return isValid;
}
{noformat}
Could you verify if that code above is like you was thinking?
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