You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Sergey Shelukhin (JIRA)" <ji...@apache.org> on 2013/02/15 20:27:13 UTC

[jira] [Updated] (HBASE-7859) errors from creating ZK offline node are not handled

     [ https://issues.apache.org/jira/browse/HBASE-7859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sergey Shelukhin updated HBASE-7859:
------------------------------------

    Priority: Minor  (was: Major)
    
> errors from creating ZK offline node are not handled
> ----------------------------------------------------
>
>                 Key: HBASE-7859
>                 URL: https://issues.apache.org/jira/browse/HBASE-7859
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Priority: Minor
>
> Quote from HBASE-7799
> {quote}
> Debugging the code and reproduced the problem.
> What i observed was
> {code}
> try {
>       ZKAssign.asyncCreateNodeOffline(watcher, state.getRegion(),
>         destination, cb, state);
>     } catch (KeeperException e) {
>       if (e instanceof NodeExistsException) {
>         LOG.warn("Node for " + state.getRegion() + " already exists");
>       } else {
>         server.abort("Unexpected ZK exception creating/setting node OFFLINE", e);
>       }
>       return false;
>     }
> return true;
> {code}
> The asyncCreateNodeOffline just always returns true because it does not wait for the callback to take action. Also the callback does not throw NodeExistsException.
> {quote}
> In short the catch block is a dead code. 

--
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