Skip to content

Remove Blob-specific ReadableStreamSource subclass#5427

Open
jasnell wants to merge 1 commit intomainfrom
jasnell/blob-use-memorystream
Open

Remove Blob-specific ReadableStreamSource subclass#5427
jasnell wants to merge 1 commit intomainfrom
jasnell/blob-use-memorystream

Conversation

@jasnell
Copy link
Collaborator

@jasnell jasnell commented Oct 28, 2025

Use the generic MemoryInputStream to provide the stream for blob data.

Part of getting ready to use the updated ReadableSource adapters

@jasnell jasnell requested a review from harrishancock October 28, 2025 21:28
@jasnell jasnell requested review from a team as code owners October 28, 2025 21:28
@codspeed-hq

This comment was marked as outdated.

@jasnell jasnell force-pushed the jasnell/blob-use-memorystream branch from 14c687b to 1006f86 Compare October 29, 2025 11:37
@jasnell jasnell requested a review from anonrig October 29, 2025 16:01
@jasnell
Copy link
Collaborator Author

jasnell commented Oct 30, 2025

Will need to investigate another issue before this can land. Moving to draft.

@jasnell jasnell marked this pull request as draft October 30, 2025 13:38
…Source

Use the generic MemoryInputStream to provide the stream for blob data
with autogate
@jasnell jasnell force-pushed the jasnell/blob-use-memorystream branch from 1006f86 to 9790596 Compare February 13, 2026 23:55
@jasnell jasnell marked this pull request as ready for review February 13, 2026 23:56
@jasnell
Copy link
Collaborator Author

jasnell commented Feb 13, 2026

@harrishancock @anonrig ... rebased and updated. Now that the other similar change has been proven to be good and stable in production we can progress with this one as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants