Skip to content

Add persistence event support and minor fixes #207

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

Closed

Conversation

sachinvmurthy
Copy link
Contributor

  1. Added support for the following persistence-related sub-events
  2. Modified deauthenticate_and_close_client_by_id() to return Status instead of ValkeyResult
  3. Updated existing tests to handle the new return type for client deauthentication

* added functioanlity to deauth and close client using ctx

Signed-off-by: sachinvmurthy <[email protected]>

* addressed comments for deauthenticate_and_close_client

Signed-off-by: sachinvmurthy <[email protected]>

* added abstraction for config get calls via context

Signed-off-by: sachinvmurthy <[email protected]>

* addressed comments

Signed-off-by: sachinvmurthy <[email protected]>

---------

Signed-off-by: sachinvmurthy <[email protected]>
Signed-off-by: sachinvmurthy <[email protected]>
@sachinvmurthy sachinvmurthy force-pushed the update/persistence_events branch from 695fc95 to 3dabda4 Compare July 18, 2025 22:25
@sachinvmurthy
Copy link
Contributor Author

conflicts not resolved properly

@sachinvmurthy sachinvmurthy deleted the update/persistence_events branch July 24, 2025 16:02
@sachinvmurthy sachinvmurthy restored the update/persistence_events branch July 24, 2025 16:04
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.

1 participant