[
https://issues.apache.org/jira/browse/HBASE-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15508530#comment-15508530
]
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
(v6.3.4#6332)
