[
https://issues.apache.org/jira/browse/KYLIN-4039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16878371#comment-16878371
]
ASF GitHub Bot commented on KYLIN-4039:
---------------------------------------
hit-lacus commented on pull request #730: Revert KYLIN-4039 because it break CI
URL: https://github.com/apache/kylin/pull/730
This reverts commit 8712a872048025d082f1f3e565f39d0f563b785f.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> ZookeeperDistributedLock may not release lock when unlock operation was
> interrupted
> -----------------------------------------------------------------------------------
>
> Key: KYLIN-4039
> URL: https://issues.apache.org/jira/browse/KYLIN-4039
> Project: Kylin
> Issue Type: Bug
> Reporter: PENG Zhengshuai
> Assignee: PENG Zhengshuai
> Priority: Major
>
> ZookeeperDistributedLock may hold the lock and not release it when the unlock
> operation was interrupted.
> Because the unlock operation contains two steps:
> 1. peekLock: get the owner of the lock
> 2. purgeLock: purge the lock if the owner of the lock is the current client.
> If the peekLock step is interrupted, the purgeLock step won't be executed.
> Thus the lock won't be released.
> Meanwhile, the lock operation should also consider the interrupt cases.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
