lhotari commented on pull request #11706:
URL: https://github.com/apache/pulsar/pull/11706#issuecomment-901665295


   > The change makes sense to me.
   > As usual we should add some test case that will save us from future 
regressions.
   > 
   > I am sorry but we need a end to end integration test for this
   
   I completely agree. The changes in the PR now fix the various issues and 
it's time to think about the test coverage to prevent regressions.


-- 
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]


Reply via email to