liaoxin01 commented on PR #63332:
URL: https://github.com/apache/doris/pull/63332#issuecomment-4550894536
This is a great find — thanks for digging into the Jetty 12 regression. A
couple of questions on the approach:
1. Are the Jetty-level maxUnconsumedRequestContentReads = -1 and the
application-level bounded drain redundant?
2. Misbehaving clients sending the body early is fundamentally a
client-side problem — they aren't implementing Expect: 100-continue correctly,
and the right fix is to push the clients to comply. Having FE actively consume
the request body by default essentially makes FE take on body traffic it was
never supposed
to handle; under high-concurrency Stream Load, this will noticeably
increase FE's network and thread pressure.
--
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]
