Skip to content

fix(framework): Fix a bug due to race conditions in time recording#6342

Merged
danieljanes merged 2 commits intomainfrom
fix-time-recording
Jan 12, 2026
Merged

fix(framework): Fix a bug due to race conditions in time recording#6342
danieljanes merged 2 commits intomainfrom
fix-time-recording

Conversation

@panh99
Copy link
Member

@panh99 panh99 commented Jan 12, 2026

The following bug happens when the main thread is trying to get the duration before the RPC handler thread saving the end time.

ERROR :     Failed to send message 36950c04aaa1b11691a5d74b320ed3a17d19850d63f82612623eca41228f32c9: Start time or end time for message ID f1dff4a2e154828676cdc9572b2dd6ed85ed6405939cdf37692be89e26f03c96 is missing.

Copy link
Member

@chongshenng chongshenng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Jan 12, 2026
@danieljanes danieljanes enabled auto-merge (squash) January 12, 2026 15:56
@danieljanes danieljanes merged commit 321105a into main Jan 12, 2026
68 of 70 checks passed
@danieljanes danieljanes deleted the fix-time-recording branch January 12, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants