Skip to content

Conversation

@SpecialYang
Copy link

@SpecialYang SpecialYang commented Dec 13, 2024

The logic for determining whether the window has expired is incorrect.

The window time range is [startTime, startTime + interval).

@LearningGp
Copy link
Collaborator

FYI #568

@LearningGp LearningGp added the area/flow-control Issues or PRs related to flow control label Jan 21, 2025
@liaomengge
Copy link

liaomengge commented Jun 3, 2025

时间窗口的范围: [startTime, startTime + interval),startTime + interval 已经是下一个窗口的起点,所以不需要使用 >= 比较,原处理逻辑貌似没啥问题。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/flow-control Issues or PRs related to flow control

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants