JasterV/granc
1
0
Fork 0
Code Issues Pull requests Releases Packages Activity

refactor: use streams for streaming responses #2

Merged
JasterV merged 1 commit from refactor/granc-core-proper-stream-handling into main 2026-03-24 00:44:30 +01:00
Conversation 0 Commits 1 Files changed 11 +253 -182
JasterV commented 2026-03-24 00:37:41 +01:00
Owner
Copy link

This PR is a continuation of: https://github.com/JasterV/granc/pull/59

At the time I implemented stream responses with a simple vector, but that is not ideal since it loads the whole stream in memory.

This improves that by using a boxed stream that can be returned to the caller for them to decide how to consume it.

This PR is a continuation of: https://github.com/JasterV/granc/pull/59 At the time I implemented stream responses with a simple vector, but that is not ideal since it loads the whole stream in memory. This improves that by using a boxed stream that can be returned to the caller for them to decide how to consume it.
JasterV force-pushed refactor/granc-core-proper-stream-handling from d37fac4676 to 43acb93dd1
All checks were successful
ci/woodpecker/pr/ci Pipeline was successful
Details
2026-03-24 00:40:06 +01:00 Compare
JasterV deleted branch refactor/granc-core-proper-stream-handling 2026-03-24 00:44:35 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
JasterV/granc!2
No description provided.