Skip to content

Fix json output memory usage - #40

Merged
steven-esser merged 4 commits into
developfrom
fix-json-output-memory-usage
Dec 21, 2017
Merged

Fix json output memory usage#40
steven-esser merged 4 commits into
developfrom
fix-json-output-memory-usage

Conversation

@steven-esser

Copy link
Copy Markdown
Contributor

No description provided.

* simplejson allows us to consume a generator during file creation
* using a generator for our calls to Delta.to_dict() significantly
  reduces memory usage of DeltaCode

Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: Steven Esser <sesser@nexb.com>

@johnmhoran johnmhoran 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.

Very nice.

@steven-esser
steven-esser merged commit 9585941 into develop Dec 21, 2017
@steven-esser
steven-esser deleted the fix-json-output-memory-usage branch December 21, 2017 23:42
arnav-mandal1234 pushed a commit to arnav-mandal1234/deltacode that referenced this pull request Mar 16, 2023
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.

2 participants