Skip to content

Implement atomic request id deduplication#565

Merged
Osvaldo Andrade (osvaldoandrade) merged 1 commit into
masterfrom
work/counters-request-dedup
Jun 23, 2026
Merged

Implement atomic request id deduplication#565
Osvaldo Andrade (osvaldoandrade) merged 1 commit into
masterfrom
work/counters-request-dedup

Conversation

@osvaldoandrade

Copy link
Copy Markdown
Collaborator

Summary

  • add optional request_id to AtomicUpdateRequest and client atomic options
  • cache successful atomic update results for retry dedup inside the Pebble shard for the 60s in-memory window
  • preserve legacy at-least-once semantics when request_id is empty

Validation

  • git diff --check
  • go test ./internal/storage/adapter/pebble ./internal/server ./pkg/client ./pkg/protocol
  • go test ./...

Closes #517

@osvaldoandrade Osvaldo Andrade (osvaldoandrade) merged commit 48f10a5 into master Jun 23, 2026
8 checks passed
@osvaldoandrade Osvaldo Andrade (osvaldoandrade) deleted the work/counters-request-dedup branch June 23, 2026 14:56
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.

Counters: request-ID dedup window for at-most-once increment

1 participant