FrankChen021 opened a new issue, #20723:
URL: https://github.com/apache/druid/issues/20723

   _This issue was generated automatically by Claude Code (Anthropic's AI 
coding agent) running a scheduled CI-triage routine on behalf of @FrankChen021. 
Analysis and suggested fixes are AI-produced; please verify before acting on 
them._
   
   **Status:** Resolved by #20717
   **Subject:** `ResponseIdentityHeaderTest` query tests (embedded-tests, `unit 
tests / unit tests(main) (25, R*,B*,P*)`): JDK 25.0.5 `HttpClient` rejects 
query response trailers
   **Failures:** 1 · **First seen:** 2026-10-24 · **Last seen:** 2026-10-24
   
   ### Root cause
   
   `setup-java` started resolving Zulu 25.0.5, which includes JDK-8383439: 
`java.net.http.HttpClient` now requires CRLF right after the last chunk, so the 
`X-Druid-Response-Complete` trailer that Brokers and Historicals sent on every 
`/druid/v2` and `/druid/v2/sql` response fails with 
`java.net.ProtocolException: CR expected`. The three query tests in 
`ResponseIdentityHeaderTest` (Broker native, Broker SQL, Historical native) 
failed deterministically on that JDK; the failing commit (#20707, license 
check) does not touch query serving, and the same job passed on Zulu 25.0.4 
earlier the same day.
   
   ### Suggested fix
   
   Merged in #20717 (`5f68651c86`): `QueryResultPusher` sends trailers only 
when the request has `TE: trailers`. The 17 and 21 jobs would hit the same 
failure on 17.0.21 / 21.0.13, so release branches need the backport (#20721).
   
   ### Occurrences
   
   Failed push-triggered master jobs only. The daily triage routine adds one 
row per new failed job.
   
   | Date | Commit | Job | Failure log | Detail | Reported in |
   |---|---|---|---|---|---|
   | 2026-10-24 | 98aa0498fe (#20707) | `unit tests / unit tests(main) (25, 
R*,B*,P*) / test-jdk25-[R*,B*,P*]` | [job 
120792864424](https://github.com/apache/druid/actions/runs/40067211857/job/120792864424)
 | 3 of 7 tests error with `ProtocolException: CR expected` in 
`ChunkedBodyParser.tryConsumeCRLF` (`sendNativeQuery:202`, `sendSqlQuery:220`); 
1/1 attempt | |
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to