qingwei91 opened a new pull request, #12166:
URL: https://github.com/apache/kafka/pull/12166

   We should sync nextTimeToEmit with wall clock on each method call to ensure 
throttling works correctly in case of clock drift.
   If we dont, then in the event of significant clock drift, throttling might 
not happen for a long time, this can hurt performance.
   
   I've added a unit test to simulate clock drift and verify my change works.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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