Hmm, I think we cannot run the verification script unless the issue is fixed?
On Thu, Nov 2, 2023 at 6:40 AM Raphael Taylor-Davies <r.taylordav...@googlemail.com.invalid> wrote: > > 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 > >> > >>