Skip to content

Instead of sync on PersistentMemberManager using CopyOnWrite structure#518

Open
suranjan wants to merge 1 commit intosnappy/masterfrom
revoke-deadlock
Open

Instead of sync on PersistentMemberManager using CopyOnWrite structure#518
suranjan wants to merge 1 commit intosnappy/masterfrom
revoke-deadlock

Conversation

@suranjan
Copy link

There was a deadlock with revoke and unblock where revoke was taking
lock on PersistentMemberManager and then requires lock on PersistenceAdvisorImpl.
The intialization thread takes lock PersistenceAdvisorImpl and then to check
whether the diskid is unblocked and requires lock on PersistentmemberManager

Changes proposed in this pull request

(Fill in the changes here)

Patch testing

(Fill in the details about how this patch was tested)

Is precheckin with -Pstore clean?

ReleaseNotes changes

(Does this change require an entry in ReleaseNotes? If yes, has it been added to it?)

Other PRs

(Does this change require changes in other projects- snappydata, spark, spark-jobserver, aqp? Add the links of PR of the other subprojects that are related to this change)

  There was a deadlock with revoke and unblock where revoke was taking
  lock on PersistentMemberManager and then requires lock on PersistenceAdvisorImpl.
  The intialization thread takes lock PersistenceAdvisorImpl and then to check
  whether the diskid is unblocked and requires lock on PersistentmemberManager
@sumwale sumwale force-pushed the snappy/master branch 2 times, most recently from 337768c to ec453a6 Compare April 8, 2022 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant