Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-06 Thread via GitHub
dajac merged PR #14931: URL: https://github.com/apache/kafka/pull/14931 -- 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:

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-06 Thread via GitHub
kirktrue commented on PR #14931: URL: https://github.com/apache/kafka/pull/14931#issuecomment-1843227108 @dajac—failing tests appear unrelated. Do we need four green builds before merging? -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue commented on PR #14931: URL: https://github.com/apache/kafka/pull/14931#issuecomment-1841792685 Four greys. Restarting build again. -- 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

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue closed pull request #14931: MINOR: ConsumerRebalanceListenerInvoker class and method visibility URL: https://github.com/apache/kafka/pull/14931 -- 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

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue commented on PR #14931: URL: https://github.com/apache/kafka/pull/14931#issuecomment-1841727258 Three yellows, one red. Restarting 爛 -- 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

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue closed pull request #14931: MINOR: ConsumerRebalanceListenerInvoker class and method visibility URL: https://github.com/apache/kafka/pull/14931 -- 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

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue commented on PR #14931: URL: https://github.com/apache/kafka/pull/14931#issuecomment-1841152541 @philipnee can you add the `ctr` and `KIP-848` tags, please? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue commented on PR #14931: URL: https://github.com/apache/kafka/pull/14931#issuecomment-1841151662 @dajac This is the shared bit pulled out from #14640 as you'd requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] MINOR: ConsumerRebalanceListenerInvoker class and method visibility [kafka]

2023-12-05 Thread via GitHub
kirktrue opened a new pull request, #14931: URL: https://github.com/apache/kafka/pull/14931 For forthcoming changes, make this class and its methods public for use outside the package. Also fix a minor logging issue to include the object in the log, not its `Optional` wrapper. -- This