shoemoney opened a new pull request, #20152:
URL: https://github.com/apache/druid/pull/20152
Broker masks 429/503 as HTML and DirectDruidClient throws JsonParseException
0x3c.
This is the second leg at DirectDruidClient.java handleResponse variant.
Same fix as first leg but for the second code path that was missed.
Fix checks status code and Content-Type, sniffs body for leading < vs { or
[, and throws QueryCapacityExceededException or QueryInterruptedException
before JSON parse. Prevents HTML being fed to the JSON parser when the broker
is throttling.
Test evidence: DirectDruidClientTest 12 tests passed RED and GREEN. Fix
verified RED to GREEN, no failures, formatter kept to one file.
--
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]
