mjsax commented on pull request #9107: URL: https://github.com/apache/kafka/pull/9107#issuecomment-751922810
About the original comment: https://github.com/apache/kafka/pull/9107#issuecomment-666749809 I am fine with those changes. About https://github.com/apache/kafka/pull/9107#issuecomment-751261181 -- that is a good point. Thanks for explaining. I guess it's a "philosophical" question if we want to allow this pattern though, or if we want to require that either `defaultBranch()` or `noDefaultBranch()` is called? -- I did consider calling `branch()` like a builder pattern, and the final `[noD|d]efaultBranch` call is basically `build()`? Curious to hear what @vvcephei thinks about it. ---------------------------------------------------------------- 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]
