cmccabe commented on PR #12856: URL: https://github.com/apache/kafka/pull/12856#issuecomment-1315706091
`controller.socket.timeout.ms` is not really the right thing here: that is used by the old ZK based controller to control timeouts for its outbound requests. In the short term, I'd suggest setting the request timeout for these send-to-broker threads to half of `broker.session.timeout.ms`. That will at least be a reasonable default here. Long term we might need a KIP to define this better. It probably makes sense as a tunable but we'd have to discuss that. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
