cloudwego / cloudwego/pilota

TBinaryProtocol returns error size when zerocopy is on and the impl doesn't use LinkedBytes

Open
#61 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
167
Forks
25
PR merge metrics
No merged PRs in 30d

Description

Currently only LinkedBytes supports zerocopy, but the length calculation can't be specialized for LinkedBytes.

This relies on https://github.com/rust-lang/rust/issues/31844, when this is stabilized, we can fix the problem.

Contributor guide

Open the contributing guide

Research direction

No file, test, or entry point is named. Trace TBinaryProtocol's length calculation and the LinkedBytes zerocopy path, then review Rust issue 31844 because the proposed fix depends on that stabilization. Done means the error size is correct when zerocopy is enabled even when the implementation does not use LinkedBytes.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.