lianetm commented on PR #14873: URL: https://github.com/apache/kafka/pull/14873#issuecomment-1848609666
Thanks for the changes @philipnee , LGTM. I would only +1 on @lucasbru suggestion of locally running the existing `PlainTextConsumer` integration test on this PR, as it is currently disabled in the CI. Also trying to get some of the max poll related ones to run (OK if in different PR, as I expect it will require some changes due to the listeners they use that we don't support yet). -- 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
