Skip to content

refactor(storage): consolidate read-path io wrappers - #17

Closed
levb wants to merge 1 commit into
mainfrom
lev-storage-rationalize
Closed

levb wants to merge 1 commit into
mainfrom
lev-storage-rationalize

Conversation

@levb

@levb levb commented Jun 1, 2026

Copy link
Copy Markdown
Owner

Replace the OpenRangeReader return type io.ReadCloser with a context-aware RangeReader and collapse scattered reader wrappers (offset/section/capture/ observable/decompress) into a single io_wrappers.go. Drops the cacheWriteThroughReader in favor of a reusable captureReader with optional drain-on-close for codecs that stop short of EOF on their source.

@levb
levb force-pushed the lev-storage-rationalize branch from e61a07f to 73dd1c2 Compare June 1, 2026 19:53
Replace the OpenRangeReader return type io.ReadCloser with a context-aware
RangeReader and collapse scattered reader wrappers (offset/section/capture/
observable/decompress) into a single io_wrappers.go. Drops the
cacheWriteThroughReader in favor of a reusable captureReader with optional
drain-on-close for codecs that stop short of EOF on their source.
@levb
levb force-pushed the lev-storage-rationalize branch from 53e81af to 042a224 Compare June 1, 2026 20:23
@levb levb closed this Jun 8, 2026
@levb
levb deleted the lev-storage-rationalize branch June 8, 2026 23:29
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.

1 participant