vasiliy-mikhailov opened a new pull request, #10554: URL: https://github.com/apache/rocketmq/pull/10554
`FaultItem.checkStamp` is never initialized, so it defaults to `0`. A freshly created fault item is therefore immediately eligible for detection in `detectByOneRound()` instead of waiting one `detectInterval` like every item that has gone through `updateFaultItem`. This makes the very first detect round fire a probe against a broker that was only just recorded. Initialize `checkStamp` to `now + detectInterval` in the constructor, matching the value `updateFaultItem` assigns. Added a regression test asserting the first `detectByOneRound()` does not probe a freshly added item. -- 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]
