Skip to content

docs(matrix): rs Memcached + File backends shipped; refresh stale capability cells (LAB-429) - #36

Merged
27Bslash6 merged 3 commits into
mainfrom
agent/winston/9f2beff8
Jul 24, 2026
Merged

docs(matrix): rs Memcached + File backends shipped; refresh stale capability cells (LAB-429)#36
27Bslash6 merged 3 commits into
mainfrom
agent/winston/9f2beff8

Conversation

@27Bslash6

Copy link
Copy Markdown
Contributor

Docs-gate companion to cachekit-io/cachekit-rs#44 (LAB-429).

  • Cache Backends: rs Memcached and File cells ✅ with feature-flag evidence (--features memcached / --features file), plus footnote ³: py-format byte compatibility (verified by running the real py FileBackend against an rs-written directory and vice versa), deliberate rs v1 gaps (no LRU eviction/size caps/mmap; single-server memcached), CI memcached container job.
  • Optional capabilities / TTL inspect: rs adds File ✅, Memcached deliberately ❌ (mirrors py). Fixed the stale Python cell — py File does implement TTLInspectableBackend today (get_ttl/refresh_ttl read/rewrite the on-disk header), and py Memcached ships refresh_ttl as a bare touch wrapper outside the protocol.
  • Distributed locking: fixed the stale rs Redis ❌ — LockableBackend for Redis landed via LAB-426 (SET NX PX + Lua compare-and-delete, live-redis CI job); note paragraph updated to match.
  • Backend-selection paragraph: Redis/Memcached/File are all wired explicitly via .backend() in rs.

…ability cells (LAB-429)

Cache Backends: rs Memcached/File flip to shipped (cargo features
memcached/file, cachekit-rs PR #44), with a footnote covering the
py-format byte compatibility, the deliberate rs v1 gaps (no eviction/
mmap; single-server memcached), and the CI container evidence.

Also corrects three cells the current code disproves:
- py File implements TTLInspectableBackend (get_ttl/refresh_ttl off the
  header) — the ❌ predated that landing; py Memcached ships refresh_ttl
  as a bare touch wrapper outside the protocol.
- rs Redis implements LockableBackend (SET NX PX + Lua compare-and-
  delete, landed via LAB-426 with a live-redis CI job) — the ❌ and the
  'TTL inspection only' note were stale.
- Reliability rows (TTL management / distributed locking) updated to
  match.

Co-authored-by: multica-agent <github@multica.ai>
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 12 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: eae72dbd-4fa7-49fc-a7ec-2c994bd27a8f

📥 Commits

Reviewing files that changed from the base of the PR and between 5ecee7f and 2992186.

📒 Files selected for processing (1)
  • sdk-feature-matrix.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/winston/9f2beff8

Comment @coderabbitai help to get the list of available commands.

27Bslash6 and others added 2 commits July 24, 2026 12:47
…emediation (LAB-429)

- rs Memcached cell: rust-memcache (async-memcached was dropped — it
  ships toxiproxy_rust/openssl as runtime deps).
- TTL inspect/refresh cells now state the true shape for BOTH py and
  rs: the protocol can't read TTLs, and each SDK ships a bare
  refresh_ttl/touch wrapper outside the capability trait.

Co-authored-by: multica-agent <github@multica.ai>
Co-authored-by: multica-agent <github@multica.ai>
@27Bslash6
27Bslash6 merged commit 05c89fb into main Jul 24, 2026
2 checks passed
@27Bslash6
27Bslash6 deleted the agent/winston/9f2beff8 branch July 24, 2026 04:47
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