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.

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

euphoria-core: earlyTriggering of merging windows #43

Open
xitep opened this issue Feb 24, 2017 · 0 comments
Open

euphoria-core: earlyTriggering of merging windows #43

xitep opened this issue Feb 24, 2017 · 0 comments

Comments

@xitep
Copy link
Contributor

xitep commented Feb 24, 2017

Early triggering of merging windows is unsound! A merging window, e.g. session window, may change its identity over time. If such a window is fired multiple times while its identity changed, receiving downstream operators have no chance associating such multiple distinct windows as originating from one.

This typically causes problems for downstream operators which do access the window for business logic reasons. We may want to perform a validation as part of translating the flow DAG and either emit a warning or have the validation fail, if the user can access a merging window in a downstream operator.

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

No branches or pull requests

1 participant