jubins commented on PR #28226: URL: https://github.com/apache/flink/pull/28226#issuecomment-4527373733
> Hi. Thank you for the follow up PR. The description says: > > > typed Java REST clients setting maxExceptions on the application endpoint > > Could u please clarify, which Java REST clients use application endpoint? AFAIK k8s operator uses `/jobs/:jobid/exceptions` and the WebUI uses raw HTTP (TypeScript). You're right, that phrasing in the description is misleading. I confirmed there are currently no typed Java clients calling. The actual fix in this PR is server-side: `ApplicationExceptionsHandler `was typed against the parent `ApplicationMessageParameters` and therefore never read the maxExceptions query parameter. This PR completes the wiring introduced in FLINK-38977 by making the handler honor `maxExceptions`, so any HTTP caller, and any future typed client built on `ApplicationExceptionsMessageParameters`, gets the expected limit behavior. -- 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]
