[PR #806] [MERGED] [Chores] Offset Allocator Test Fix & Docs Fix #1835

New issue

Closed

opened 2026-10-29 21:30:44 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 21:30:44 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/Mooncake/pull/806
Author: @ykwd
Created: 9/4/2025
Status: ✅ Merged
Merged: 9/5/2025
Merged by: @xiaguan

Base: main ← Head: offset-allocator-test-fix

📝 Commits (2)

a87e5d8 Fix unit test
27ccebf fix docs

📊 Changes

4 files changed (+3 additions, -18 deletions)

View changed files

📝 doc/en/architecture.md (+1 -1)
📝 doc/zh/architecture.md (+1 -1)
📝 docs/source/design/architecture.md (+1 -1)
📝 mooncake-store/tests/offset_allocator_test.cpp (+0 -15)

📄 Description

Removed a test case that, in rare cases, could attempt to allocate an excessively large block of memory, potentially causing the unit test to fail or exhausting system memory resources, which was triggered here: https://github.com/kvcache-ai/Mooncake/actions/runs/17458641225/job/49577922428
Fix a minor docs issue.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/Mooncake/pull/806 **Author:** [@ykwd](https://github.com/ykwd) **Created:** 9/4/2025 **Status:** ✅ Merged **Merged:** 9/5/2025 **Merged by:** [@xiaguan](https://github.com/xiaguan) **Base:** `main` ← **Head:** `offset-allocator-test-fix` --- ### 📝 Commits (2) - [`a87e5d8`](https://github.com/kvcache-ai/Mooncake/commit/a87e5d8bcb9926d0204d759b0a171037ae1459b8) Fix unit test - [`27ccebf`](https://github.com/kvcache-ai/Mooncake/commit/27ccebf2c90bd208093d9d4b1d04848629ad00c1) fix docs ### 📊 Changes **4 files changed** (+3 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `doc/en/architecture.md` (+1 -1) 📝 `doc/zh/architecture.md` (+1 -1) 📝 `docs/source/design/architecture.md` (+1 -1) 📝 `mooncake-store/tests/offset_allocator_test.cpp` (+0 -15) </details> ### 📄 Description 1. Removed a test case that, in rare cases, could attempt to allocate an excessively large block of memory, potentially causing the unit test to fail or exhausting system memory resources, which was triggered here: https://github.com/kvcache-ai/Mooncake/actions/runs/17458641225/job/49577922428 2. Fix a minor docs issue. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-29 21:30:44 +00:00