fix: Fix error on empty range requests

[nameexhaustion]

• Fixes Generic S3 error raised when reading parquet file from S3 with DELTA_BINARY_PACKED encoding #23832

Return empty Bytes object if the range is empty. I've also opened an issue upstream at apache/arrow-rs-object-store#453.

Note that I cannot reproduce the Python MRE in the issue, but I've tested this in the Rust side:

---- cloud::polars_object_store::tests::test_ stdout ----
[crates/polars-io/src/cloud/polars_object_store.rs:552:13] store.1.get_range(&object_path_from_str("iris.parquet").unwrap(), 0..0).await = Ok(
    b"",
)
[crates/polars-io/src/cloud/polars_object_store.rs:559:13] store.1.get_range(&object_path_from_str("iris.parquet").unwrap(), i32::MAX as usize..i32::MAX as usize).await = Ok(
    b"",
)
[crates/polars-io/src/cloud/polars_object_store.rs:566:13] store.1.get_ranges_sort(&object_path_from_str("iris.parquet").unwrap(), &mut
[0..0, 0..0]).await = Ok(
    {
        0: MemSlice {
            slice: [],
            inner: Bytes(
                b"",
            ),
        },
    },
)

[codecov]

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.39%. Comparing base (4b0f8dd) to head (b467f5a).
⚠️ Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/polars-io/src/cloud/polars_object_store.rs	66.66%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #23844      +/-   ##
==========================================
+ Coverage   81.38%   81.39%   +0.01%     
==========================================
  Files        1653     1653              
  Lines      224519   224525       +6     
  Branches     2873     2873              
==========================================
+ Hits       182718   182761      +43     
+ Misses      41087    41050      -37     
  Partials      714      714              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[tustvold]

Should this be returning an empty range even if the file doesn't exist?

[nameexhaustion]

Should this be returning an empty range even if the file doesn't exist?

For our use case, it's not really a concern if the file doesn't actually exist. That being said, the approach we've taken here might not necessarily be a suitable default for the object store library itself.