Skip to content

Add mutex for saving process#219

Draft
xepozz wants to merge 3 commits into
masterfrom
mutex
Draft

Add mutex for saving process#219
xepozz wants to merge 3 commits into
masterfrom
mutex

Conversation

@xepozz

@xepozz xepozz commented Jun 18, 2023

Copy link
Copy Markdown
Member
Q A
Is bugfix? ✔️/❌
New feature? ✔️/❌
Breaks BC? ✔️/❌
Fixed issues comma-separated list of tickets # fixed by the PR, if any

@what-the-diff

what-the-diff Bot commented Jun 18, 2023

Copy link
Copy Markdown

PR Summary

  • Added new dependency
    A new dependency has been added to manage package installation.
  • Imported Synchronizer in Debugger
    The Synchronizer class is now being used in the Debugger for better performance.
  • Prevented race conditions
    Wrapped the flush function with Synchronizer to ensure only one process can execute it at a time, avoiding potential race conditions.

vjik
vjik requested changes Jun 29, 2023
View reviewed changes

@vjik vjik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need adapt tests to this change.

@xepozz xepozz marked this pull request as draft July 9, 2023 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vjik vjik vjik requested changes
@samdark samdark samdark approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xepozz @samdark @vjik