[
https://issues.apache.org/jira/browse/HBASE-4247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13100320#comment-13100320
]
Akash Ashok commented on HBASE-4247:
------------------------------------
In HConnectionManager.java under the subclass HConnectionManagerImplementation
{code}
@Override
public void abort(final String msg, Throwable t) {
if (t instanceof KeeperException.SessionExpiredException) {
try {
LOG.info("This client just lost it's session with ZooKeeper, trying" +
" to reconnect.");
resetZooKeeperTrackers();
LOG.info("Reconnected successfully. This disconnect could have been" +
" caused by a network partition or a long-running GC pause," +
" either way it's recommended that you verify your environment.");
return;
} catch (ZooKeeperConnectionException e) {
LOG.error("Could not reconnect to ZooKeeper after session" +
" expiration, aborting");
t = e;
}
}
if (t != null) LOG.fatal(msg, t);
else LOG.fatal(msg);
this.closed = true;
}
{code}
If we call close() instead of this tests are failing( one of them being
TestMergeTools ). I was wondering if some change should be made out here.
Somehow I feel its not right set this.closed out here? Or if this is some
special case and could be left untouched.
Thanks
> Add isAborted method to the Abortable interface
> -----------------------------------------------
>
> Key: HBASE-4247
> URL: https://issues.apache.org/jira/browse/HBASE-4247
> Project: HBase
> Issue Type: Task
> Reporter: Akash Ashok
> Assignee: Akash Ashok
> Priority: Minor
> Fix For: 0.94.0
>
>
> Add a new method isAborted() to the Abortable interface
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
