Skip to content

fix: fix TestWindowState race test#3997

Merged
ngjaying merged 4 commits intolf-edge:masterfrom
Yisaer:fix_race_test
Feb 2, 2026
Merged

fix: fix TestWindowState race test#3997
ngjaying merged 4 commits intolf-edge:masterfrom
Yisaer:fix_race_test

Conversation

@Yisaer
Copy link
Collaborator

@Yisaer Yisaer commented Feb 2, 2026

No description provided.

Signed-off-by: Song Gao <disxiaofei@163.com>
@codecov
Copy link

codecov bot commented Feb 2, 2026

Codecov Report

❌ Patch coverage is 54.43038% with 36 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.81%. Comparing base (85d4f19) to head (36d3aae).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
internal/topo/node/window_inc_agg_op.go 67.19% 20 Missing and 1 partial ⚠️
internal/topo/node/window_inc_agg_event_op.go 0.00% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3997      +/-   ##
==========================================
+ Coverage   71.34%   71.81%   +0.47%     
==========================================
  Files         457      460       +3     
  Lines       52899    53183     +284     
==========================================
+ Hits        37737    38190     +453     
+ Misses      12247    12047     -200     
- Partials     2915     2946      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Song Gao <disxiaofei@163.com>
Signed-off-by: Song Gao <disxiaofei@163.com>
@Yisaer Yisaer marked this pull request as ready for review February 2, 2026 05:18
@Yisaer Yisaer requested a review from ngjaying February 2, 2026 05:18
Signed-off-by: Song Gao <disxiaofei@163.com>
@ngjaying ngjaying merged commit 2507eb1 into lf-edge:master Feb 2, 2026
60 of 61 checks passed
ngjaying added a commit to ngjaying/kuiper that referenced this pull request Feb 4, 2026
Signed-off-by: Song Gao <disxiaofei@163.com>
Co-authored-by: Yisaer <disxiaofei@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants