Skip to content

QQ: fix consumer timeout bug#15805

Merged
michaelklishin merged 1 commit intomainfrom
qq-consumer-timeout-bug-attempt-2
Mar 25, 2026
Merged

QQ: fix consumer timeout bug#15805
michaelklishin merged 1 commit intomainfrom
qq-consumer-timeout-bug-attempt-2

Conversation

@kjnilsson
Copy link
Copy Markdown
Contributor

@kjnilsson kjnilsson commented Mar 23, 2026

where an infinity timer would be emitted for the same command
but after an actual timer leader to no timer.

Move to a model where consumer timers are handled by the aux
handler after each eval batch to reduce the number of timers
emitted.

@kjnilsson kjnilsson added this to the 4.3.0 milestone Mar 23, 2026
@kjnilsson kjnilsson force-pushed the qq-consumer-timeout-bug-attempt-2 branch 2 times, most recently from 7b49367 to 4f7fe19 Compare March 24, 2026 13:32
where an infinity timer would be emitted for the same command
but after an actual timer leader to no timer.

Move to a model where consumer timers are handled by the aux
handler after each eval batch to reduce the number of timers
emitted.
@kjnilsson kjnilsson force-pushed the qq-consumer-timeout-bug-attempt-2 branch from 4f7fe19 to 99f16a0 Compare March 24, 2026 14:41
@kjnilsson kjnilsson requested a review from ansd March 24, 2026 14:41
@kjnilsson kjnilsson marked this pull request as ready for review March 24, 2026 14:41
@kjnilsson kjnilsson requested a review from mkuratczyk March 24, 2026 14:52
@michaelklishin michaelklishin modified the milestones: 4.3.0, 4.4.0 Mar 25, 2026
@michaelklishin michaelklishin merged commit aa33f1d into main Mar 25, 2026
184 checks passed
@michaelklishin michaelklishin deleted the qq-consumer-timeout-bug-attempt-2 branch March 25, 2026 01:03
michaelklishin added a commit that referenced this pull request Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants