[ https://issues.apache.org/jira/browse/HBASE-17251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15730765#comment-15730765 ]
Duo Zhang commented on HBASE-17251: ----------------------------------- Fine. Let me commit the v1 patch. > Add a timeout parameter when locating region > -------------------------------------------- > > Key: HBASE-17251 > URL: https://issues.apache.org/jira/browse/HBASE-17251 > Project: HBase > Issue Type: Sub-task > Components: asyncclient, Client > Affects Versions: 2.0.0 > Reporter: Duo Zhang > Assignee: Duo Zhang > Fix For: 2.0.0 > > Attachments: HBASE-17251-v1.patch, HBASE-17251.patch > > > Now we always use the default timeout configured for zk and meta. > I think it is reasonable to always use the same timeout when accessing zk and > meta as the result will be shared by lots of threads. If we could do a > successful fetching then the result will be in cache so it does not make > sense to set the timeout to a very small value when fetching. > But I think it is also important to let the user request finish in time even > if the user can only get a timeout exception. We should not block a user > request longer than operation timeout. > So I think we could add a timeout parameter to the region locate method, and > if the location can not be fetched in time, we will just finished the > returned CompletableFuture with a timeout exception, but the actual fetching > operation can still go on and use its own timeout config. -- This message was sent by Atlassian JIRA (v6.3.4#6332)