Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Lock] Fix expired lock not cleaned in ZooKeeper #32075

Merged
merged 1 commit into from Jun 17, 2019

Conversation

jderusse
Copy link
Member

Q A
Branch? 4.3
Bug fix? yes
New feature? ,p
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #31426
License MIT
Doc PR NA

Following #32071 for 4.3 branch

context:
When a lock is acquired BUT not as fast as expected, a LockExpiredException is thrown.
Issue is, that the lock is not removed which avoid other process to acquire that lock.

This PR clean state of store when a LockExpiredException is triggered in PDO and ZooKeepeer.

@jderusse jderusse changed the title Fix Expiring lock in PDO and ZooKeeper [Lock] Fix expired lock not cleaned in PDO and ZooKeeper Jun 17, 2019
@nicolas-grekas
Copy link
Member

Rebase + retarget needed :)

@jderusse jderusse changed the title [Lock] Fix expired lock not cleaned in PDO and ZooKeeper [Lock] Fix expired lock not cleaned in ZooKeeper Jun 17, 2019
@jderusse jderusse changed the base branch from 4.4 to 4.3 June 17, 2019 19:09
@jderusse
Copy link
Member Author

done

@nicolas-grekas nicolas-grekas added this to the 4.3 milestone Jun 17, 2019
@nicolas-grekas
Copy link
Member

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit 4f808ef into symfony:4.3 Jun 17, 2019
nicolas-grekas added a commit that referenced this pull request Jun 17, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

[Lock] Fix expired lock not cleaned in ZooKeeper

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | ,p
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #31426
| License       | MIT
| Doc PR        | NA

Following #32071 for 4.3 branch

context:
When a lock is acquired BUT not as fast as expected, a LockExpiredException is thrown.
Issue is, that the lock is not removed which avoid other process to acquire that lock.

This PR clean state of store when a LockExpiredException is triggered in PDO and ZooKeepeer.

Commits
-------

4f808ef Fix Expiring lock in PDO and ZooKeeper
@jderusse jderusse deleted the fix-expired-lock-43 branch August 2, 2019 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants