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

fix: make JsonDiffer thread safe #683

Merged
merged 3 commits into from
Dec 27, 2023
Merged

fix: make JsonDiffer thread safe #683

merged 3 commits into from
Dec 27, 2023

Conversation

rajaniraog
Copy link
Contributor

Concurrent execution of JsonDiffer() on multiple threads causes ArgumentOutOfRangeException during Dispose().
Create new instance of JsonPatchDeltaFormatter for every call.
Please refer to this issue: #682

Concurrent execution of JsonDiffer() on multiple threads causes
ArgumentOutOfRangeException during Dispose().
Create new instance of JsonPatchDeltaFormatter for every call.
@buehler buehler merged commit fb2f914 into buehler:main Dec 27, 2023
3 checks passed
buehler added a commit that referenced this pull request Jan 17, 2024
Concurrent execution of JsonDiffer() on multiple threads causes
ArgumentOutOfRangeException during Dispose().
Create new instance of JsonPatchDeltaFormatter for every call.
Please refer to this issue:
#682

---------

Co-authored-by: AVEVA\rajani.rao <rajani.rao@aveva.com>
Co-authored-by: Christoph Bühler <buehler@users.noreply.github.com>
buehler added a commit that referenced this pull request Jan 17, 2024
Concurrent execution of JsonDiffer() on multiple threads causes
ArgumentOutOfRangeException during Dispose().
Create new instance of JsonPatchDeltaFormatter for every call.
Please refer to this issue:
#682

---------

Co-authored-by: AVEVA\rajani.rao <rajani.rao@aveva.com>
Co-authored-by: Christoph Bühler <buehler@users.noreply.github.com>
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.

None yet

2 participants