hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "nkeywal (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-6356) printStackTrace in FSUtils
Date Mon, 09 Jul 2012 15:11:34 GMT

    [ https://issues.apache.org/jira/browse/HBASE-6356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13409552#comment-13409552
] 

nkeywal commented on HBASE-6356:
--------------------------------

Yes, we're not allowed to throw. I wonder if it's better to log.warn or to stop here. Today
when it happens it returns false, so the easy option is to just log and saying that we keep
the backward compatibility this way...
                
> 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: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message