Skip to content

Conversation

aricart
Copy link
Member

@aricart aricart commented Sep 19, 2025

This adds support for configuring the persistence mode for a stream.

This adds support for configuring the persistence mode for a stream.

Signed-off-by: Alberto Ricart <[email protected]>
…tence options

- Introduced `PersistMode` enum with `Default` and `Async` options for configuring stream persistence.
- Added `persist_mode` property to stream configurations for flexibility in persistence model settings.

Signed-off-by: Alberto Ricart <[email protected]>
…tence options

- Introduced `PersistMode` enum with `Default` and `Async` options for configuring stream persistence.
- Added `persist_mode` property to stream configurations for flexibility in persistence model settings.

Signed-off-by: Alberto Ricart <[email protected]>
Copy link
Member

@bruth bruth left a comment

Choose a reason for hiding this comment

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

Suggestions. Not sure how much in the weeds you want to get, but felt like a bit more clarity could help.

@aricart
Copy link
Member Author

aricart commented Sep 19, 2025

@bruth yes changes are good - accepted

@aricart aricart requested a review from bruth September 19, 2025 20:31
@aricart aricart merged commit 43e3e4a into main Sep 19, 2025
24 checks passed
@aricart aricart deleted the consistency-models branch September 19, 2025 20:42
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