Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

Pull upstream #10

Merged
merged 5 commits into from
May 6, 2024
Merged

Pull upstream #10

merged 5 commits into from
May 6, 2024

Conversation

antiguru
Copy link
Member

@antiguru antiguru commented May 6, 2024

No description provided.

antiguru and others added 5 commits April 23, 2024 10:44
* Allow joins to produce arbitrary containers

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 <antiguru@gmail.com>

* Remove `join_traces_core`

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

---------

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>
* Remove use of ValOwned

* Respond to feedback
* Split default merge batcher

This change splits the default merge batcher implementation into a type
that maintains the outer part of its algorithm, specifically knows how
to maintain chains, and an inner part that knows how to maintain the
individual batches in chains. The benefit is that the outer part does
not need to know about the contents of the containers it holds on to
because that's encapsulated in the inner trait's implementation.

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

* Undo some changes, rip out old columnated batcher

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

* Address feedback

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

* Formatting and renaming

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

* Address feedback

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>

---------

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>
join_traces can produce arbitrary containers.

---------

Signed-off-by: Moritz Hoffmann <antiguru@gmail.com>
@antiguru antiguru merged commit a3f0f31 into MaterializeInc:master May 6, 2024
0 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants