Technoboy- commented on PR #21238: URL: https://github.com/apache/pulsar/pull/21238#issuecomment-1732546765
> > we already not support these versions > > @Technoboy- here we have one question: do we still keep compatible with those versions? These tests do sanity checks that new version clients can talk to old version servers, this is part of our compatibility promise. > > We also have `pulsar-client-1x-base`. If we don't need to ensure compatibility use <= 2.8, we may not need to maintain those modules also. > > What do you think? The reason for this PR is that when I was working on #21221, I found that some scripts in the test were no longer universal. I wanted to change them, but found that they had little practical significance. Therefore, I want to delete all the related tests in this PR. If it goes according to your opinion, indeed, they should all be deleted. However, we may need to discuss in the mail. -- 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]
