Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: race condition while publishing wm inside reduce #1599

Merged
merged 18 commits into from
Apr 3, 2024
Merged

fix: race condition while publishing wm inside reduce #1599

merged 18 commits into from
Apr 3, 2024

Conversation

yhl25
Copy link
Contributor

@yhl25 yhl25 commented Mar 25, 2024 •
edited by vigith
Loading

partially fixes #1607

@yhl25
fix: race condition while publishing wm inside reduce
b60c631 
Signed-off-by: Yashash H L <[email protected]>
@yhl25
update reduce_udf
369e3db 
Signed-off-by: Yashash H L <[email protected]>
@yhl25
lint
c114ffd 
Signed-off-by: Yashash H L <[email protected]>
@yhl25
update yaml
5551a27 
Signed-off-by: Yashash H L <[email protected]>
@yhl25
improve ctx handling for reduce
1d1fcc9 
Signed-off-by: Yashash H L <[email protected]>
@yhl25
fix writes
ba63bea 
Signed-off-by: Yashash H L <[email protected]>
@yhl25 yhl25 marked this pull request as ready for review March 26, 2024 15:03
@yhl25 yhl25 marked this pull request as draft March 26, 2024 15:03
@vigith vigith linked an issue Mar 31, 2024 that may be closed by this pull request
An old message popped up issue during multiple reduce vertices and pod restarts #1607
Closed
@yhl25 yhl25 marked this pull request as ready for review April 2, 2024 16:33
@yhl25 yhl25 merged commit 3dbba4f into main Apr 3, 2024
23 checks passed
@yhl25 yhl25 deleted the pnf branch April 3, 2024 04:29
yhl25 added a commit that referenced this pull request Apr 3, 2024
@yhl25 @vigith
fix: race condition while publishing wm inside reduce (#1599)
45032fc 
Signed-off-by: Yashash H L <[email protected]>
Signed-off-by: Vigith Maurice <[email protected]>
Co-authored-by: Vigith Maurice <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vigith vigith vigith approved these changes

@whynowy whynowy Awaiting requested review from whynowy whynowy is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

An old message popped up issue during multiple reduce vertices and pod restarts
2 participants
@yhl25 @vigith