Skip to content

Add support for partial decode / multithreading mode choice#112

Merged
csvance merged 9 commits into
mainfrom
partial-decode
Jun 4, 2026
Merged

Add support for partial decode / multithreading mode choice#112
csvance merged 9 commits into
mainfrom
partial-decode

Conversation

@csvance

@csvance csvance commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator
  • gRPC proxies need to be able to do partial decoding on raw inputs and pass along the raw buffer as the output. This matters especially with large protobufs which are seen with KServe protocol.
  • Add choice for the user between sticky and non sticky tasks. sticky has better throughput / IO performance, but prevents proper multithreading. Fixes Sticky Test Failing on Nightly #114

@codecov-commenter

codecov-commenter commented Jun 2, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.59%. Comparing base (a99aa4a) to head (4cb8949).

Files with missing lines Patch % Lines
src/gRPC.jl 71.42% 2 Missing ⚠️
src/Utils.jl 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
- Coverage   90.96%   90.59%   -0.38%     
==========================================
  Files           7        7              
  Lines         531      542      +11     
==========================================
+ Hits          483      491       +8     
- Misses         48       51       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@csvance csvance changed the title Add support for partial decode Add support for partial decode / multithreading modle choice Jun 2, 2026
@csvance csvance changed the title Add support for partial decode / multithreading modle choice Add support for partial decode / multithreading mode choice Jun 2, 2026
@csvance csvance merged commit 0f62b98 into main Jun 4, 2026
13 checks passed
csvance referenced this pull request Jun 9, 2026
Adds an optional `token` keyword to the service client. When set, every
request (unary and streaming) carries an `authorization: Bearer <token>`
header; when left as `nothing` no header is sent, so existing behaviour is
unchanged.

The token field is threaded through the generated `*_Client` constructors,
gRPCServiceClient, and the request builder. The Go test server gains a unary
interceptor that validates the header, and a new "Bearer Authentication"
testset asserts that the correct token is accepted, a wrong token is rejected
with UNAUTHENTICATED, and the default (no-token) client is unaffected.

Closes #56
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.

2 participants