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.

Sign up for GitHub

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

Jump to bottom

fix concurrent collection modification issue #62

Merged

wenxi-zeng merged 1 commit into main from wenxi/fix-concurrent-collection-modified-issue

Sep 18, 2023

Contributor

wenxi-zeng commented Sep 15, 2023 •

edited

attempt to address issue mentioned in #61 caused by concurrent access of collection (dictionary and list)

replace Dictionary with ConcurrentDictionary
replace List with ConcurrentList
update Serialization.NET to latest that makes JsonObject and JsonArray thread safe while enumerating
update Unity LifecyclePlugin sample to use ConcurrentList to be thread safe on read.


          fix concurrent collection modification issue

b69a419

wenxi-zeng requested a review from MichaelGHSeg

September 15, 2023 21:20

MichaelGHSeg approved these changes

View reviewed changes

wenxi-zeng merged commit eca3213 into main

4 checks passed

wenxi-zeng deleted the wenxi/fix-concurrent-collection-modified-issue branch

September 18, 2023 15:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment