Skip to content

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