universalmind303 opened a new pull request, #4841:
URL: https://github.com/apache/arrow-rs/pull/4841
# Which issue does this PR close?
Closes #4839
# Rationale for this change
<!--
Why are you proposing this change? If this is already explained clearly in
the issue then this section is not needed.
Explaining clearly why changes are proposed helps reviewers understand your
changes and offer better suggestions for fixes.
-->
# What changes are included in this PR?
If a range was requested and the server returned a 200 instead of a 206, we
error out as it means that the server didn't fulfill the request.
# Are there any user-facing changes?
this could cause some backwards incompatibility with previously working
queries no longer working as we are now a bit stricter about what we are
expecting.
Some examples that would no longer work
- a server _could_ satisfy the range request, but due to misconfiguration is
still returning a `200`.
- the server never satisfied the range request & we fetched the entire data
multiple times.
<!--
If there are user-facing changes then we may require documentation to be
updated before approving the PR.
-->
<!---
If there are any breaking changes to public APIs, please add the `breaking
change` label.
-->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
