Skip to content

Support huge JSON data#2286

Draft
diegolovison wants to merge 1 commit into
Hyperfoil:masterfrom
diegolovison:configure_threshold
Draft

Support huge JSON data#2286
diegolovison wants to merge 1 commit into
Hyperfoil:masterfrom
diegolovison:configure_threshold

Conversation

@diegolovison
Copy link
Copy Markdown
Contributor

Changes proposed

Support huge JSON data

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@lampajr lampajr self-requested a review February 26, 2025 08:19
Copy link
Copy Markdown
Member

@lampajr lampajr left a comment

Choose a reason for hiding this comment

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

Hi @diegolovison, I did some tests and the changes are actually fixing the problem of uploading a big run but TBH I don't think we should set such defaults here.

Both can be added where needed at runtime (and if and only if are needed):

  • Server side: quarkus.http.limits.max-body-size=300M can be passed as JVM option when starting the server (i.e., we can do that in out internal lab for instance)
  • Client side: org.jboss.resteasy.client.jaxrs.engines.fileUploadInMemoryThreshold can be set before creating the HorreumClient, so that if users need that can do on their own and have a way to configure it. If we hardcode that users CANNOT override that value

@diegolovison
Copy link
Copy Markdown
Contributor Author

Server side: quarkus.http.limits.max-body-size=300M can be passed as JVM option when starting the server (i.e., we can do that in out internal lab for instance)

what is the limit on the database?

@lampajr
Copy link
Copy Markdown
Member

lampajr commented Feb 26, 2025

Server side: quarkus.http.limits.max-body-size=300M can be passed as JVM option when starting the server (i.e., we can do that in out internal lab for instance)

what is the limit on the database?

Don't remember exactly (need to check), but we should try to keep the json objects as smaller as possible for performance reasons.. therefore I would prefer to keep a lower limit by default - that's why I don't think increasing the quarkus max body limit is the good choice here (at least for the defaults)

@lampajr
Copy link
Copy Markdown
Member

lampajr commented Mar 3, 2025

As agreed this is going to be updated to become a new guide section or paragraph showing how users can deal with big JSON data. @diegolovison converting this back to draft but feel free to close this one and open a new one if you consider so.

@lampajr lampajr marked this pull request as draft March 3, 2025 09:49
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