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

[Bug][Go SDK]: Plumb the allowed lateness. It is getting ignored in graphx/translate.go #22474

Closed
riteshghorse opened this issue Jul 27, 2022 · 1 comment · Fixed by #22476
Closed

[Bug][Go SDK]: Plumb the allowed lateness. It is getting ignored in graphx/translate.go #22474

riteshghorse opened this issue Jul 27, 2022 · 1 comment · Fixed by #22476
Assignees
@riteshghorse
Labels
bug done & done Issue has been reviewed after it was closed for verification, followups, etc. go P2

Comments

@riteshghorse
Copy link
Contributor

What happened?

Right now, users of Go SDK can set the AllowedLateness for windows but it gets ignored in graphx/translate.go. It always sets it to 0 instead of setting it from WindowingStrategy.

Issue Priority

Priority: 2

Issue Component

Component: sdk-go
@riteshghorse
Copy link
Contributor Author

.take-issue

@riteshghorse riteshghorse mentioned this issue Jul 27, 2022
Merged
4 tasks
@chamikaramj chamikaramj added the done & done Issue has been reviewed after it was closed for verification, followups, etc. label Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@riteshghorse riteshghorse

Labels
bug done & done Issue has been reviewed after it was closed for verification, followups, etc. go P2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Go SDK]: Plumb allowed lateness to execution riteshghorse/beam
2 participants
@chamikaramj @riteshghorse