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

Allow joins to produce arbitrary containers #477

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

antiguru
Copy link
Member

Add a join_traces_core function that allows the caller to specify the output stream container type. The existing join_traces function forces it to be vectors, by virtue of wrapping the output in a collection.

Add a `join_traces_core` function that allows the caller to specify the
output stream container type. The existing `join_traces` function forces
it to be vectors, by virtue of wrapping the output in a collection.

Signed-off-by: Moritz Hoffmann <[email protected]>
@antiguru antiguru requested a review from frankmcsherry April 23, 2024 01:55
Copy link
Member

@frankmcsherry frankmcsherry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, but ideally we could merge join_traces and join_traces_core: the more general one is great, and the slightly less general one can probably just be skipped (as you note, it is just a .as_collection() call away).

Signed-off-by: Moritz Hoffmann <[email protected]>
@antiguru antiguru merged commit 3e81b8b into TimelyDataflow:master Apr 23, 2024
7 checks passed
@antiguru antiguru deleted the join_traces_core branch April 23, 2024 14:44
This was referenced 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.

2 participants