devinbost commented on issue #6202: [Issue 6200] [Issue 6198] Reduce flakiness of tests URL: https://github.com/apache/pulsar/pull/6202#issuecomment-587816675 @sijie @merlimat I need some guidance about how I should split up this PR into smaller PRs since it fixes quite a lot of issues. I'm choosing to omit a few specific flaky tests from this PR that are deeper issues (and may involve actual code fixes rather than just fixing tests.) In this PR, all of the changes are fixing tests, not actual code. I could break it up into: 1. Fixes of issues caused by test timeouts being too short. (The majority of changes in this PR are in this category.) 2. Fixes of issues caused by hanging tests resulting from no timeouts. 3. Fixes of Prometheus test issues (since those were more extensive changes to the tests). 4. Changes that introduced awaitility library. (Only used in a couple of places to avoid the use of Thread.sleep.) 5. Fixes involving concurrency issues on topic names. (There were a few in this category.) Thoughts?
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
