Skip to content

[refactor] Use batch_encode_into/batch_decode_from for Yuanrong backend#110

Merged
0oshowero0 merged 1 commit into
Ascend:mainfrom
dpj135:remove_pack_from_yr
May 30, 2026
Merged

[refactor] Use batch_encode_into/batch_decode_from for Yuanrong backend#110
0oshowero0 merged 1 commit into
Ascend:mainfrom
dpj135:remove_pack_from_yr

Conversation

@dpj135
Copy link
Copy Markdown
Contributor

@dpj135 dpj135 commented May 30, 2026

Decription

Adjust logics of yuanrong_client based on #107 .

  • raplace pack_into/encode with batch_encode_into.
  • replace unpack_from/decode with batch_decode_from

@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

dpj135, thanks for your pull request. All authors of the commits have signed the CLA. 👍

…code_from

Signed-off-by: dpj135 <958208521@qq.com>
@dpj135 dpj135 force-pushed the remove_pack_from_yr branch from 3faae0b to 36d58d2 Compare May 30, 2026 08:21
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

dpj135, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@dpj135 dpj135 marked this pull request as ready for review May 30, 2026 08:37
@0oshowero0 0oshowero0 requested a review from Copilot May 30, 2026 08:42
@0oshowero0 0oshowero0 changed the title [Fix] Replace pack/unpack and encode/decode with batch_encode_into/batch_decode_from [refactor] Use batch_encode_into/batch_decode_from for Yuanrong backend May 30, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Yuanrong general KV zero-copy serialization path to use the shared batch serialization helpers introduced in #107.

Changes:

  • Replaces local pack/unpack helpers with batch_encode_into / batch_decode_from.
  • Adjusts Yuanrong zero-copy set/get flow to allocate DataSystem buffers through the batch encoder.
  • Updates the Yuanrong zero-copy test mocks to patch shared serialization entry points.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
transfer_queue/storage/clients/yuanrong_client.py Uses shared batch serialization helpers for general KV zero-copy storage and retrieval.
tests/test_yuanrong_client_zero_copy.py Updates test serialization/deserialization mocks for the new batch helper path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0oshowero0 0oshowero0 merged commit 7fc6ef9 into Ascend:main May 30, 2026
8 checks passed
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.

4 participants