mjsax commented on pull request #11211: URL: https://github.com/apache/kafka/pull/11211#issuecomment-1068535207
> If there is, cache, then it hits the underneath caching layer again. I felt this could have been inverted, no? Not sure if I can follow. What you exactly mean? > And also, if we don't fix caching in this PR, then the results returned with and w/o caching would be different which might be confusing. I agree that we should fix it, but we can still can do it in a follow up PR (it's easier to keep a single PR focused on one thing). As long as we ensure that both PRs go into the same release, we avoid this confusing situation for the user. -- 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]
