[PR #1205] [MERGED] [TE] Memorize batched transfer status

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

kvcache-ai/Mooncake#2070

bsakaguchi commented

2026-10-29 21:32:33 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/Mooncake/pull/1205
Author: @alogfans
Created: 12/12/2025
Status: ✅ Merged
Merged: 12/17/2025
Merged by: @alogfans

Base: main ← Head: fix-duplicated-query

📝 Commits (4)

46d21a4 [TE] Memorize batched transfer status
75242ae Use atomic type
13cddae Merge success with is_finished
7e5ec03 reformat code

📊 Changes

2 files changed (+17 additions, -4 deletions)

View changed files

📝 mooncake-transfer-engine/include/transport/transport.h (+6 -3)
📝 mooncake-transfer-engine/src/multi_transport.cpp (+11 -1)

📄 Description

Description

We memorize the complete status of batched operations. If it's completed, no further query required. This avoids duplicated getTransferStatus. It could also resolve #1094.

Type of Change

Types
- Bug fix
- New feature
  - Transfer Engine
  - Mooncake Store
  - Mooncake EP
  - Integration
  - P2P Store
  - Python Wheel
- Breaking change
- CI/CD
- Documentation update
- Other

How Has This Been Tested?

Tested in local environment.

Checklist

I have performed a self-review of my own code.
I have updated the documentation.
I have added tests to prove my changes are effective.

_{🔄 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/1205 **Author:** [@alogfans](https://github.com/alogfans) **Created:** 12/12/2025 **Status:** ✅ Merged **Merged:** 12/17/2025 **Merged by:** [@alogfans](https://github.com/alogfans) **Base:** `main` ← **Head:** `fix-duplicated-query` --- ### 📝 Commits (4) - [`46d21a4`](https://github.com/kvcache-ai/Mooncake/commit/46d21a40dd1bb0d3a2884fd9a0da5fed66c8e1aa) [TE] Memorize batched transfer status - [`75242ae`](https://github.com/kvcache-ai/Mooncake/commit/75242ae6ed97acc81a554088fc713b79aa40cef5) Use atomic type - [`13cddae`](https://github.com/kvcache-ai/Mooncake/commit/13cddaeed933d51520143f051fece6ee01798dfd) Merge success with is_finished - [`7e5ec03`](https://github.com/kvcache-ai/Mooncake/commit/7e5ec03f4fc19489a8cb50ec4c66d96cbbaf87e9) reformat code ### 📊 Changes **2 files changed** (+17 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `mooncake-transfer-engine/include/transport/transport.h` (+6 -3) 📝 `mooncake-transfer-engine/src/multi_transport.cpp` (+11 -1) </details> ### 📄 Description ## Description We memorize the complete status of batched operations. If it's completed, no further query required. This avoids duplicated getTransferStatus. It could also resolve #1094. ## Type of Change * Types - [X] Bug fix - [ ] New feature - [ ] Transfer Engine - [ ] Mooncake Store - [ ] Mooncake EP - [ ] Integration - [ ] P2P Store - [ ] Python Wheel - [ ] Breaking change - [ ] CI/CD - [ ] Documentation update - [ ] Other ## How Has This Been Tested? Tested in local environment. ## Checklist - [X] I have performed a self-review of my own code. - [X] I have updated the documentation. - [X] I have added tests to prove my changes are effective. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

closed this issue
added the
pull-request
label

Rows
Columns

[PR #1205] [MERGED] [TE] Memorize batched transfer status #2070