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

Introduce inspect_core operator #418

Merged
merged 2 commits into from
Aug 25, 2021

Conversation

frankmcsherry
Copy link
Member

Generalizes inspect to present both data events and frontier advancement events. Fixes TimelyDataflow/differential-dataflow#338

@frankmcsherry frankmcsherry merged commit 94102cd into TimelyDataflow:master Aug 25, 2021
@antiguru
Copy link
Member

I like the idea! Otoh, I'm not too fond of using Result to disambiguate frontiers and data. It seems like stretching the concept. As an alternative, why not provide two functions, or use a specialized enumeration?

@danhhz
Copy link
Contributor

danhhz commented Aug 26, 2021

I like the idea! Otoh, I'm not too fond of using Result to disambiguate frontiers and data. It seems like stretching the concept. As an alternative, why not provide two functions, or use a specialized enumeration?

https://docs.rs/timely/0.12.0/timely/dataflow/operators/to_stream/enum.Event.html is even already pretty close to the specialized enum you'd need

@github-actions github-actions bot mentioned this pull request Oct 29, 2024
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.

Deterministic completion of a batch (frontier)
3 participants