vvcephei commented on pull request #11582: URL: https://github.com/apache/kafka/pull/11582#issuecomment-998165138
Thanks, all! This is all really good feedback, and we clearly have some opportunities to tighten up both the API and internals. I will go through the follow-on tickets that are filed in Jira and mark them as 3.2 blockers. That way, we can be sure that we will either address the API issues before the release or pull IQv2 from the release. Immediately after merging, I will follow up with a PR to remove `QueryResult#swapResult` from the public API. I think both the fact that it's an instance method and its name were misleading, but after reflection it also doesn't really need to be part of the caller-facing result class at all. -- 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
