tombentley commented on pull request #9878: URL: https://github.com/apache/kafka/pull/9878#issuecomment-764836936
Incremental first would be my preference. It doesn't preclude doing something else in the future. If we improved `KafkaFuture` then I would argue that new admin methods should still use it, for consistency. Allowing a mix of KafkaFuture and CompletionStage just makes the API inconsistent, makes Admin2 feel more necessary than it needs to be, and doesn't add any additional functionality. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
