[
https://issues.apache.org/jira/browse/HBASE-10472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13894036#comment-13894036
]
Enis Soztutar commented on HBASE-10472:
---------------------------------------
I guess we need this for checking the meta location from zk right? Do we need
that to be interruptible as well? I guess we might need it if we ever want to
go meta replicas.
We are not setting the interrupted flag for the thread in
ReplicationPeersZKImpl, ZKLeaderManager. Not sure whether they are needed.
Can we get away with some of the changes, if we keep the original
ZKUtil.getData() and have ZKUtil.getDataInterruptible() ?
Other than those it looks good.
> Manage the interruption in ZKUtil#getData
> -----------------------------------------
>
> Key: HBASE-10472
> URL: https://issues.apache.org/jira/browse/HBASE-10472
> Project: HBase
> Issue Type: Bug
> Components: Client, master, regionserver
> Affects Versions: 0.98.0, 0.99.0
> Reporter: Nicolas Liochon
> Assignee: Nicolas Liochon
> Fix For: 0.98.1, 0.99.0
>
> Attachments: 10472.v1.patch, 10472.v2.patch
>
>
> Many, but not all, methods in ZKUtil partly hides the interruption: they
> return null or something like that. Many times, this will result in a NPE, or
> something undefined.
> This jira is limited to the getData to keep things small enough (it's used in
> hbase-client code).
> The code is supposed to behave at least 'as well as before', or better
> (hopefully).
> It could be included in a later .98 release (98.1) and in .99
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
