guozhangwang commented on PR #13319: URL: https://github.com/apache/kafka/pull/13319#issuecomment-1450565316
I checked the latest failure on `shouldPauseStandbyTaskAndNotTransit..` and found out that it's because of the test condition itself is timing dependent. What it wants to verify is the following sequence: `task1 added, task2 added, and then task1 paused`. In this case the `transitToUpdateStandby` should be called once. But it is possible that the following sequence can also happen: `task1 added, task1 paused, task2 added` (because the updater thread can interleave its runOnce with the `addTask` call), in which case the `transitToUpdateStandby` could be called twice. I've relaxed that condition and I think this test should not fail anymore. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
