Skip to content

Supercache dev - #24

Merged
zchuango merged 3 commits into
zchuango:supercache_devfrom
liangxu2000:supercache_dev
Jun 21, 2026
Merged

zchuango merged 3 commits into
zchuango:supercache_devfrom
liangxu2000:supercache_dev

Conversation

@liangxu2000

@liangxu2000 liangxu2000 commented Jun 17, 2026

Copy link
Copy Markdown

Description

Module

  • Transfer Engine (mooncake-transfer-engine)
  • Mooncake Store (mooncake-store)
  • Mooncake EP (mooncake-ep)
  • Mooncake PG (mooncake-pg)
  • Integration (mooncake-integration)
  • P2P Store (mooncake-p2p-store)
  • Python Wheel (mooncake-wheel)
  • Common (mooncake-common)
  • Mooncake RL (mooncake-rl)
  • CI/CD
  • Docs
  • Other

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Breaking change
  • Documentation update
  • Performance improvement
  • Other

How Has This Been Tested?

Test commands:

# Example: bash scripts/run_ci_test.sh

Test results:

  • Unit tests pass
  • Integration tests pass (if applicable)
  • Manual testing done (describe below)

Checklist

  • I have performed a self-review of my own code
  • I have formatted my code using ./scripts/code_format.sh
  • I have run pre-commit run --all-files and all hooks pass
  • I have updated the documentation (if applicable)
  • I have added tests to prove my changes are effective
  • For changes >500 LOC: I have filed an RFC issue

AI Assistance Disclosure

  • No AI tools were used
  • AI tools were used (specify below)

- Add avg/p50/p90 to latency output (previously only min/max/p99/p999/p9999)
- Replace ops/sec with keys/sec and queries/sec for clearer semantics
  (keys = individual data units, queries = API calls get_into/batch_get_into)
- Ensure latency is per-query: one latency sample per API call,
  not per-key in batch mode
Use floating-point division to avoid truncation when sum/count
is small, which caused avg to display as 0.00.
Aggregate() was not accumulating avg_latency_ns across intervals,
so overall.avg_latency_ns stayed at initial value 0.
Now uses weighted average (sum of avg*count / total count).
@zchuango
zchuango merged commit ae1277a into zchuango:supercache_dev Jun 21, 2026
2 of 3 checks passed
yyyuanhao426-hash pushed a commit to yyyuanhao426-hash/Mooncake that referenced this pull request Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants