Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@driusan
Copy link
Collaborator

@driusan driusan commented Apr 5, 2024

This updates the candidate_list module to use format=binary format instead of format=json. For large projects with many candidates the encoding of the data to JSON can use up all the server's memory and then crash with an out-of-memory error.

Since the format=binary format is a line-based streamable format, it does not require loading the entire result set into the server's memory to encode it to JSON before transmitting it to the client and larger data sets can be loaded into LORIS.

This updates the candidate_list module to use format=binary
format instead of format=json. For large projects with many
candidates the encoding of the data to JSON can use up all
the server's memory and then crash with an out-of-memory error.

Since the format=binary format is a line-based streamable format,
it does not require loading the entire result set into memory before
transmitting it to the client and larger data sets can be loaded into
LORIS.
@kongtiaowang kongtiaowang self-assigned this Apr 18, 2024
@kongtiaowang kongtiaowang added the Passed manual tests PR has been successfully tested by at least one peer label Apr 18, 2024
Copy link
Contributor

@kongtiaowang kongtiaowang left a comment

Choose a reason for hiding this comment

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

LGTM

@driusan driusan merged commit aca44a6 into aces:main Apr 18, 2024
@ridz1208 ridz1208 added this to the 26.0.0 milestone Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants