modelcontextprotocol / modelcontextprotocol/java-sdk
Streamable HTTP client: slow reception of large SSE tool responses (fromLineSubscriber line-assembly bottleneck)
@chemicL y travaille déjà.
Depuis le 3/7/2026.
- Langage dominant
- Java
- Étoiles
- 3.7k
- Forks
- 1.1k
- Merge moyen
- 1 j 15 h
- PR mergées (30 j)
- 9
Description
Summary
When a tool returns a large response over the Streamable HTTP client transport, the client takes ~5s to receive a ~4 MB body that curl/HttpClient.ofString() reads in ~0.4s. The bottleneck is the client-side SSE body reading in ResponseSubscribers.sseToBodySubscriber, not the server, network, or JSON parsing.
Environment
io.modelcontextprotocol.sdk:mcp-core/mcp2.0.0 (latest)- JDK 25, Reactor (via SDK)
- Server: Spring AI 2.0.0
mcp-spring-webmvc(HttpServletStreamableServerTransportProvider) returning a single large SSEmessageevent — one compact-JSONdata:line (~4.17 MB) - Client:
HttpClientStreamableHttpTransport(McpSyncClient.callTool)
Measurements (steady-state, 3 runs)
| Path | Time to receive ~4.17 MB |
|---|---|
McpSyncClient.callTool (this SDK) |
~5,300 ms |
Same payload via curl / HttpClient BodyHandlers.ofString() |
~0.4–0.75 s |
| Jackson parse of the received JSON | < 70 ms |
→ ~10–13× slower than a plain one-shot read of the identical bytes.
Root cause analysis
The response body is effectively a single huge data: line (compact JSON has no newlines). ResponseSubscribers.sseToBodySubscriber uses HttpResponse.BodySubscribers.fromLineSubscriber(...); assembling that one ~4 MB line through the JDK line subscriber is the cost.
Things I tried:
- Changing
SseLineSubscriberdemand fromupstream().request(1)torequest(Long.MAX_VALUE)→ no improvement (so it isn't per-line backpressure). - Replacing the body subscriber with a streaming byte-level SSE parser (
BodySubscribers.fromSubscriber, unbounded demand, accumulateByteBuffers, split on\n\nevent boundaries, decode each event once) → ~0.4 s (≈13×).
Important constraint (must stay streaming)
A whole-body ofString read fixes the speed but breaks progress: the server interleaves notifications/progress on the same POST response SSE stream before the final result. So the fix must remain a streaming parser that emits each SSE event as its boundary arrives (a byte-level parser does this while still avoiding the line-assembly cost).
Questions
- Is there a recommended approach/workaround for large tool responses on the client that we're missing?
- Would you accept a PR replacing
fromLineSubscriberwith a streaming byte-level SSE parser inResponseSubscribers.sseToBodySubscriber(preserving incremental event emission)? - Is this related to #844 (opt-in
application/jsonresponse mode)? That would avoid SSE framing on the server side, but clients receiving SSE responses would still benefit from this fix.
Happy to open a PR with the streaming parser + a benchmark if that's welcome.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Évaluation
Cette issue n'a pas encore été évaluée.