[GitHub] [kafka] showuon commented on pull request #11746: MINOR: Use timeout config in KafkaAdminClient

2022-02-11 Thread GitBox
showuon commented on pull request #11746: URL: https://github.com/apache/kafka/pull/11746#issuecomment-1036126819 @dengziming , I'm thinking if we should add some comment to this 1 hour setting, so that it won't confuse other developers next time. WDYT? -- This is an automated message fr

[GitHub] [kafka] showuon commented on pull request #11746: MINOR: Use timeout config in KafkaAdminClient

2022-02-11 Thread GitBox
showuon commented on pull request #11746: URL: https://github.com/apache/kafka/pull/11746#issuecomment-1036034155 @dajac , looks like you're right. [KAFKA-5324](https://issues.apache.org/jira/browse/KAFKA-5324) changed the value from `reqeust timeout config value` to 1 hour, to have per-ca