Skip to content

feat: user events handler partial recovery#794

Open
aintnostressin wants to merge 1 commit intomainfrom
feat/user-handler-fix-state
Open

feat: user events handler partial recovery#794
aintnostressin wants to merge 1 commit intomainfrom
feat/user-handler-fix-state

Conversation

@aintnostressin
Copy link
Copy Markdown
Contributor

Prepares user handlers for partial recovery.

  • Reorders the user DEL handler to delete the graph node last (after Redis/search cleanup), so that a retry after partial failure re-enters the CreatedOrDeleted branch and re-runs idempotent Redis DEL/ZREM ops before finally removing the graph node.
  • Replaces UserDetails::delete() with remove_from_index_multiple_json() which is a pure Redis DEL — idempotent and safe to re-run without needing the graph node.
  • Adds two integration tests: full idempotent retry (expects SkipIndexing) and graph-last partial-failure recovery.

@aintnostressin aintnostressin requested review from ok300 and tipogi April 3, 2026 16:37
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