zigarn commented on PR #12434: URL: https://github.com/apache/kafka/pull/12434#issuecomment-1197803586
Thanks for this thorough feedback @C0urante! Fixed the `checkstyle/import-control.xml`, did revert too much after a lot of testing and importing new packages. About the `DefaultHandler`, looks like it's currently not working anyway and didn't find a way to make it work... From my debugging, looks like the context handler is putting the request in `HANDLED` status and next handlers are skipped. Maybe an issue by mixing simple handler and context handlers? Maybe `DefaultHandler` should be set as an inner handler of the context handlers? I have to say that it's not easy to get into all this handler mechanism of Jetty. I'll add the handling of autoclosable client and responses. -- 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]
