fred-ro commented on PR #17086: URL: https://github.com/apache/kafka/pull/17086#issuecomment-2333571961
I had a look at those consumer constructors and I do not think that just swapping lines will do the job as some exceptions might been raised before. So solutions would be to remove the final and set a default Optional.empty() value or to remove the Optional by that good old nullable field. I am not aware of any coding policy in Kafka about that. Any guidance or opinion ? -- 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]
