Skip to content

[Feat] Modify task datatype of AsuClient to spsc ring queue - #1306

Merged
Infinite666 merged 6 commits into
ModelEngine-Group:feature_26h1from
Fengli5355:br_AsuClient_SpscClient
Aug 31, 2026
Merged

[Feat] Modify task datatype of AsuClient to spsc ring queue#1306
Infinite666 merged 6 commits into
ModelEngine-Group:feature_26h1from
Fengli5355:br_AsuClient_SpscClient

Conversation

@Fengli5355

@Fengli5355 Fengli5355 commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Purpose

This PR optimizes data structure of AsuClient tasks.

Modifications

  1. Replace deque with SpscRingQueue
  2. Add some compiling options for kv-test
  3. kv-test bench adds --count option to set IO number to certain value
  4. Reserve space for router variables
  5. Add config MaxInflightTasks for client/transport seperately

Test

All unit tests are passed.
An offline inference is passed with fake/aiv provider.
Kv-test runs bench command normally with --count option.

Comment thread ucm/transport/kv/asu/client/src/asu_client_impl.cpp Outdated
@Fengli5355
Fengli5355 requested a review from ygwpz as a code owner August 29, 2026 07:42
@Fengli5355
Fengli5355 force-pushed the br_AsuClient_SpscClient branch from 368a1d9 to e88a1c5 Compare August 31, 2026 02:38
@Infinite666
Infinite666 merged commit 828de0a into ModelEngine-Group:feature_26h1 Aug 31, 2026
8 checks passed
Infinite666 pushed a commit to Infinite666/unified-cache-management that referenced this pull request Aug 31, 2026
…ine-Group#1306)

## Purpose
This PR optimizes data structure of AsuClient tasks.

## Modifications 
1. Replace deque with SpscRingQueue
2. Add some compiling options for kv-test
3. kv-test bench adds --count option to set IO number to certain value
4. Reserve space for router variables
5. Add config MaxInflightTasks for client/transport seperately

## Test
All unit tests are passed.
An offline inference is passed with fake/aiv provider.
Kv-test runs bench command normally with --count option.
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.

3 participants