Stephen Yuan Jiang commented on HBASE-16587:

[~mbertozzi], could you put the patch into RB for easy code review?  

For V3 patch, I don't see your override the newly added holdLock() and 
hasLock() in the test procedure.  (the logic of checking both holdLock() and 
hasLock() seems over complicated, but I have not see an issue here)

> Procedure v2 - Cleanup suspended proc execution
> -----------------------------------------------
>                 Key: HBASE-16587
>                 URL: https://issues.apache.org/jira/browse/HBASE-16587
>             Project: HBase
>          Issue Type: Sub-task
>          Components: proc-v2
>    Affects Versions: 2.0.0
>            Reporter: Matteo Bertozzi
>            Assignee: Matteo Bertozzi
>             Fix For: 2.0.0
>         Attachments: HBASE-16587-v0.patch, HBASE-16587-v1.patch, 
> HBASE-16587-v2.patch, HBASE-16587-v3.patch
> for procedures like the assignment or the lock one we need to be able to hold 
> on locks while suspended. At the moment the way to do that is up to the proc 
> implementation. This patch moves the logic to the base Procedure and 
> ProcedureExecutor.

This message was sent by Atlassian JIRA

Reply via email to