Skip to content

MCR-3445 increase request buffer size in solrj #2538

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.

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

Draft
wants to merge 3 commits into
base: 2025.06.x
Choose a base branch
from

Conversation

rsteph-de
Copy link
Member

@rsteph-de rsteph-de force-pushed the issues/MCR-3445_increase_request_buffer_size_in_solrj branch from 7d94ff6 to 8d9928f Compare May 21, 2025 17:24
@rsteph-de rsteph-de requested review from sebhofmann and yagee-de May 21, 2025 17:25

Method mSetRequestBufferSize = jettyHttpClient.getClass()
.getDeclaredMethod("setRequestBufferSize", int.class);
mSetRequestBufferSize.invoke(jettyHttpClient, 511 * 1024);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this specific hard-coded value? Could be a configuration property for applications that need an even larger buffer size.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good idea. we can even use the exististence of the property to decide if the modifications are necessary.
(see commit)

@rsteph-de rsteph-de marked this pull request as draft July 1, 2025 13:37
@rsteph-de rsteph-de force-pushed the issues/MCR-3445_increase_request_buffer_size_in_solrj branch from 8d9928f to cf6064a Compare July 10, 2025 14:47
@rsteph-de rsteph-de force-pushed the issues/MCR-3445_increase_request_buffer_size_in_solrj branch from cf6064a to 704fbe6 Compare July 10, 2025 14:52
@rsteph-de rsteph-de changed the base branch from 2024.06.x to 2025.06.x July 11, 2025 10:47
@rsteph-de rsteph-de requested a review from toKrause July 11, 2025 12:45
@rsteph-de rsteph-de marked this pull request as ready for review July 11, 2025 12:46
@rsteph-de
Copy link
Member Author

I am setting this back to draft.
I added another PR which handles alto word coordinates in a separate Solr atomic update request

@rsteph-de rsteph-de marked this pull request as draft July 12, 2025 15:21
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