Skip to content

redis: add live collector for metrics and per-command heat - #3

Merged
briheet merged 2 commits into
mainfrom
briheet/redis-adapter
Aug 20, 2026
Merged

briheet merged 2 commits into
mainfrom
briheet/redis-adapter

Conversation

@briheet

@briheet briheet commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Adds internal/adapters/redis that observes a running Redis server over its public protocol (no source build or injection required):

  • Synthetic StaticGraph of well-known commands (analysis/) for TUI attribution
  • INFO memory/stats/cpu/clients -> model.RuntimeMetrics (metrics/)
  • INFO commandstats -> per-command calls/usec heat profile (trace/)
  • go-redis based Collector; Start pings + enables latency-tracking

Registered as the "redis" target in the factory; engine skips directory validation so the arg is a host:port address. Adds redis to the flake devShell and go-redis/v9 dependency.

Perf: cached the known-command set and cut profile allocations ~271x (trace.Decode 988KB/617 allocs -> 3.6KB/117 allocs per call).

Adds internal/adapters/redis that observes a running Redis server over its
public protocol (no source build or injection required):
- Synthetic StaticGraph of well-known commands (analysis/) for TUI attribution
- INFO memory/stats/cpu/clients -> model.RuntimeMetrics (metrics/)
- INFO commandstats -> per-command calls/usec heat profile (trace/)
- go-redis based Collector; Start pings + enables latency-tracking

Registered as the "redis" target in the factory; engine skips directory
validation so the arg is a host:port address. Adds redis to the flake
devShell and go-redis/v9 dependency.

Perf: cached the known-command set and cut profile allocations ~271x
(trace.Decode 988KB/617 allocs -> 3.6KB/117 allocs per call).
@briheet
briheet force-pushed the briheet/redis-adapter branch from 4672b6c to 146091f Compare August 20, 2026 04:44
@briheet
briheet merged commit 5f1018d into main Aug 20, 2026
1 check passed
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