philipnee commented on PR #14680: URL: https://github.com/apache/kafka/pull/14680#issuecomment-1802364240
Hi @kirktrue Thanks for taking time reviewing the PR: Just to quickly respond to your comment. The BackgroundEventProcessor is already invoked inside poll() and if the event is added to the queue and the process method is added similar to the above, that should do the trick. The callback also needs to be invoked on all commit calls, if we decide to stick with the current behavior, which I assume we do. (Even though I personally would like to conslidate all invocation on poll only, to make things consistent). -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
