michaeljmarshall opened a new pull request, #20279: URL: https://github.com/apache/pulsar/pull/20279
Discussed on mailing list here: https://lists.apache.org/thread/ky2bkzlz93njx3ntnvkpd0l77qzzgcmv Fixes: https://github.com/apache/pulsar/issues/19088 https://github.com/apache/pulsar/issues/16450 https://github.com/apache/pulsar/issues/15422 https://github.com/apache/pulsar/issues/11428 ### Motivation I am putting this PR together to move the mailing list conversation along. If we decide to keep this code, I will close this PR. Copied from @eolivelli on the mailing list thread: > There are many flaky tests about that feature and I believe that it has never been used in Production by anyone, because it happened a few times that we did some changes in the regular Dispatcher and introduced bugs on the StreamingDispacther (usually manifested as flaky tests) ### Modifications * Remove all `StreamingDispatcher` code, tests, and configuration. ### Verifying this change It should be sufficient to see the tests pass. ### Does this pull request potentially affect one of the following parts: This affects configuration. ### Documentation - [x] `doc-not-needed` There are no references to "streaming dispatcher" or "streamingdispatcher" in our github `pulsar-site` repo, which indicates to me that no docs need to be updated. ### Matching PR in forked repository PR in forked repository: https://github.com/michaeljmarshall/pulsar/pull/46 -- 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: commits-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
