Skip to content

Conversation

@jessegeens
Copy link
Contributor

@jessegeens jessegeens commented Oct 16, 2025

Up to now, Reva supported Range requests only when the requested content was copied into Reva's memory. This has now been improved: the ranges can be propagated upto the storage provider, which only returns the requested content, instead of first copying everything into memory and then only returning the requested ranges

@update-docs
Copy link

update-docs bot commented Oct 16, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@jessegeens jessegeens force-pushed the fix/range-requests branch 3 times, most recently from 7633101 to 4373759 Compare October 16, 2025 15:10
@jessegeens jessegeens marked this pull request as ready for review October 16, 2025 15:12
diocas
diocas previously approved these changes Oct 17, 2025
@jessegeens jessegeens force-pushed the fix/range-requests branch 3 times, most recently from 7b42832 to dcf725a Compare October 17, 2025 09:59
@jessegeens jessegeens changed the title [WIP] Proper support of Range requests Proper support of Range requests Oct 17, 2025
@jessegeens jessegeens marked this pull request as draft October 17, 2025 11:43
@jessegeens jessegeens force-pushed the fix/range-requests branch 6 times, most recently from e740e4d to 403de93 Compare October 17, 2025 12:16
@jessegeens jessegeens marked this pull request as ready for review October 17, 2025 12:21
Up to now, Reva supported Range requests only when the requested content
was copied into Reva's memory. This has now been improved: the ranges
can be propagated upto the storage provider, which only returns the
requested content, instead of first copying everything into memory and
then only returning the requested ranges
@glpatcern glpatcern merged commit 08bc6f3 into master Oct 17, 2025
14 checks passed
@jessegeens jessegeens deleted the fix/range-requests branch December 11, 2025 13:07
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.

4 participants