Skip to content

Add output reset API skeleton#6018

Merged
gz merged 2 commits intomainfrom
delta-flushing-part1
Apr 12, 2026
Merged

Add output reset API skeleton#6018
gz merged 2 commits intomainfrom
delta-flushing-part1

Conversation

@gz
Copy link
Copy Markdown
Contributor

@gz gz commented Apr 11, 2026

Adds API skeleton/necessary platform support for #5980.

Makes it easier to test it because the platform changes are available afterwards.

Describe Incompatible Changes

None, reset will return a 501 not implemented error.

@gz gz requested a review from ryzhyk April 11, 2026 06:15
Signed-off-by: feldera-bot <feldera-bot@feldera.com>
Copy link
Copy Markdown

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

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

Two blockers, see inline.

Comment thread crates/pipeline-manager/src/api/endpoints/pipeline_interaction.rs
Comment thread crates/pipeline-manager/src/api/endpoints/pipeline_interaction.rs
Copy link
Copy Markdown
Contributor

@ryzhyk ryzhyk left a comment

Choose a reason for hiding this comment

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

Reset may possibly be useful for non-snapshot-and-follow operators as well.
But I think this is ok to experimentation purposes.

@gz gz added this pull request to the merge queue Apr 12, 2026
Merged via the queue into main with commit 46a2b6d Apr 12, 2026
1 check passed
@gz gz deleted the delta-flushing-part1 branch April 12, 2026 03:22
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.

4 participants