Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

feat(si-layer-cache): impl rebaser requests work queue stream#3470

Merged
si-bors-ng[bot] merged 1 commit intomainfrom
fnichol/layerdb-rebaser-work-queue
Mar 25, 2024
Merged

feat(si-layer-cache): impl rebaser requests work queue stream#3470
si-bors-ng[bot] merged 1 commit intomainfrom
fnichol/layerdb-rebaser-work-queue

Conversation

@fnichol
Copy link
Copy Markdown
Contributor

@fnichol fnichol commented Mar 22, 2024

This is an initial implementation of a work queue stream of rebaser requests, suitable for use by the rebaser service.

Note that it's not yet clear if additional work queue streams follow this pattern, if the code is extractable into the service-related crates, or if some other code de-duplication is possible. At the moment, the newly added Rust types are doing a lot of heavy lifting to ensure correctness of behavior and type safety.

Also note that this NATS Jetstream setup may require further testing and refinement to confirm the desired stream sourcing behavior.

This is an initial implementation of a work queue stream of rebaser
requests, suitable for use by the rebaser service.

Note that it's not yet clear if additional work queue streams follow
this pattern, if the code is extractable into the service-related
crates, or if some other code de-duplication is possible. At the moment,
the newly added Rust types are doing a *lot* of heavy lifting to ensure
correctness of behavior and type safety.

Also note that this NATS Jetstream setup may require further testing and
refinement to confirm the desired stream sourcing behavior.

Signed-off-by: Fletcher Nichol <fletcher@systeminit.com>
@fnichol
Copy link
Copy Markdown
Contributor Author

fnichol commented Mar 22, 2024

bors try

si-bors-ng Bot added a commit that referenced this pull request Mar 22, 2024
@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 22, 2024

try

Build succeeded:

@adamhjk
Copy link
Copy Markdown
Contributor

adamhjk commented Mar 25, 2024

bors merge

@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 25, 2024

Build succeeded:

@si-bors-ng si-bors-ng Bot merged commit 030a772 into main Mar 25, 2024
@si-bors-ng si-bors-ng Bot deleted the fnichol/layerdb-rebaser-work-queue branch March 25, 2024 15:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants