ijuma commented on PR #18218: URL: https://github.com/apache/kafka/pull/18218#issuecomment-2552207997
> Will we be able to delete LazyDownConversionRecords and the remaining down conversion code after this PR lands? Yes. This and a few other things are what I meant by: > Future PRs will remove conditional code that is no longer needed (some of that has been done in KafkaApis, but only what was required due to the schema changes). We can probably do that in master only as it does not change behavior. > I think we'll be leaning on the test suite for this change :) Yeah, I think that will ensure the remaining protocol versions work correctly. For the clients that are no longer supported, I will test them manually and considering what we can do via the system tests. -- 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]
