gosonzhang commented on PR #25349: URL: https://github.com/apache/pulsar/pull/25349#issuecomment-4219018870
> > What suggestions do you have? > > The initial idea was to keep the execution logic consistent before and after the test case modification. Personally, I also think that constructing a test API to execute the `waitForCursorRewindingRefCnf++` operation is not a good approach, but I haven't found a better one. > > I don't know yet until I understand exactly what the test is currently doing and how the problem is triggered in production code. After understanding that it's easier to recommend option. I asked some questions in another comment. The testReplicatorExpireMsgAsync() and testReplicatorClearBacklog() methods related to waitForCursorRewindingRefCnf have been modified. -- 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]
