Aah, that was a mistake introduced in a test in [1], where it was incorrectly relying on a particular ordering when listing a directory. I've created a PR that should fix the issue [2].

As this is purely a testing oversight, I am not sure this warrants cutting another RC unless you feel otherwise?

[1]: https://github.com/apache/arrow-rs/pull/5020/files#diff-e0de0bcc9edd75e6cb6bb15dda180797a0c809b1e01b23cc2189f76409a1c2f5R1426
[2]: https://github.com/apache/arrow-rs/pull/5026

On 02/11/2023 13:31, Andrew Lamb wrote:
When I ran the verification script (on mac x86-46) it failed one of the
tests:

```
---- local::tests::invalid_path stdout ----
thread 'local::tests::invalid_path' panicked at src/local.rs:1389:9:
assertion `left == right` failed
   left: [Path { raw: "directory/child.txt" }, Path { raw: "💀" }]
  right: [Path { raw: "💀" }, Path { raw: "directory/child.txt" }]
note: run with `RUST_BACKTRACE=1` environment variable to display a
backtrace


failures:
     local::tests::invalid_path
```



On Thu, Nov 2, 2023 at 7:48 AM Raphael Taylor-Davies
<r.taylordav...@googlemail.com.invalid> wrote:

Hi,

I would like to propose a release of Apache Arrow Rust Object
Store Implementation, version 0.8.0.

This release candidate is based on commit:
ad211fe324d259bf9fea1c43a3a82b3c833f6d7a [1]

The proposed release tarball and signatures are hosted at [2].

The changelog is located at [3].

Please download, verify checksums and signatures, run the unit tests,
and vote on the release. There is a script [4] that automates some of
the verification.

!! Please note you will need to use the latest version of the
verification scripts on master !!

This is the result of a doctest issue when compiling with only the
default features [5]

The vote will be open for at least 72 hours.

[ ] +1 Release this as Apache Arrow Rust Object Store
[ ] +0
[ ] -1 Do not release this as Apache Arrow Rust Object Store because...

[1]:

https://github.com/apache/arrow-rs/tree/ad211fe324d259bf9fea1c43a3a82b3c833f6d7a
[2]:

https://dist.apache.org/repos/dist/dev/arrow/apache-arrow-object-store-rs-0.8.0-rc1
[3]:

https://github.com/apache/arrow-rs/blob/ad211fe324d259bf9fea1c43a3a82b3c833f6d7a/object_store/CHANGELOG.md
[4]:

https://github.com/apache/arrow-rs/blob/master/object_store/dev/release/verify-release-candidate.sh
[5]: https://github.com/apache/arrow-rs/issues/5025


Reply via email to