mjsax commented on PR #22541: URL: https://github.com/apache/kafka/pull/22541#issuecomment-4675451225
Thanks for the PR. One high level comment/question: for "classic" groups, there is only `session.timeout.ms` (which is a client side config). With the new KIPs, we get two configs, a _broker_ config (which act as default for all groups if not overwritten), and the _group_ config (that each group can set individually -- so this one, is "closer" to the previous client side config). Thus, I am wondering if we should only bump the priority to HIGH for the _group_ configs? But keep the _broker_ one a medium? -- 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]
