Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[candidate_list] Make candidate_list use LORIS streamable binary format #9195

Merged
merged 1 commit into from
Apr 18, 2024

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 undergone proper testing 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
28 checks passed
@ridz1208 ridz1208 added this to the 26.0.0 milestone Jun 6, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Passed Manual Tests PR has undergone proper testing by at least one peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants