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

Duo Zhang commented on HBASE-19978:
-----------------------------------

The UT is not good enough yet... We need multiple tables instead of one table 
with multiple regions here, since all the AssignProcedures will be in the same 
queue, so we can easily pick the procedure for recovering meta, even without 
priority.

Will go out in the next two days so upload the patch here for reviewing first. 
Will modify the UT when come back.

[~stack] FYI.

> The keepalive logic is incomplete in ProcedureExecutor
> ------------------------------------------------------
>
>                 Key: HBASE-19978
>                 URL: https://issues.apache.org/jira/browse/HBASE-19978
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Major
>             Fix For: 2.0.0-beta-2
>
>         Attachments: HBASE-19978.patch
>
>
> The worker thread will just exit after keep alive time, and we never add it 
> back. The only way to add it back is through the stuck checker, this is not 
> correct. Here we should start new worker thread if it is under the core pool 
> size and there are pending procedures.
> For now the default keep alive time is Long.MAX_VALUE which means no timeout 
> so no problem, but we do allow users to set it so we need to fix it.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to