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.

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 OOM with JsonSerializer #2529

Merged
merged 3 commits into from
Jul 10, 2021
Merged

Fix OOM with JsonSerializer #2529

merged 3 commits into from
Jul 10, 2021

Conversation

shargon
Copy link
Member

@shargon shargon commented Jul 9, 2021

Close #2527

@shargon shargon requested a review from erikzhang July 9, 2021 08:32
@shargon shargon mentioned this pull request Jul 9, 2021
Closed
vang1ong7ang
vang1ong7ang previously approved these changes Jul 9, 2021
View reviewed changes
src/neo/SmartContract/JsonSerializer.cs Outdated Show resolved Hide resolved
@erikzhang erikzhang merged commit cf3a865 into neo-project:master Jul 10, 2021
@vang1ong7ang vang1ong7ang mentioned this pull request Jul 10, 2021
Closed
@shargon shargon deleted the fix-json-serializer branch July 12, 2021 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@erikzhang erikzhang erikzhang approved these changes

@vang1ong7ang vang1ong7ang vang1ong7ang left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

B**OOM** in jsonSerialize
3 participants
@shargon @erikzhang @vang1ong7ang