shoemoney commented on PR #20151: URL: https://github.com/apache/druid/pull/20151#issuecomment-5500680477
Agreed on both. I'll fold #20152 into this PR so the chunk-classification state lives in one place, and address the two findings here: restrict the 503 shortcut to HTML bodies so a JSON `SERVICE_UNAVAILABLE` still parses as before, and carry the handler exception through `handleChunk` so a later-chunk failure isn't replaced by the generic `ChannelException` — with a multi-chunk regression test for that path. I'll close #20152 once this carries its diff. -- 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]
