Good call Ruihang. I remember we used to have this toolchain file when
we were still in the main arrow repo. I will take a look into that.

On Wed, Aug 11, 2021 at 5:36 PM Wayne Xia <waynest...@gmail.com> wrote:
>
> Hi QP,
>
> When running this script I noticed that this might be because I was not
> using a stable toolchain when testing.
> Those failures occur with nightly (which is my default toolchain). And
> everything works fine after switching to stable 1.54.
> So I think it's ok from my side to vote +1.
>
> BTW, I think we can add a toolchain file [1] to datafusion repo.
>
> [1]: https://rust-lang.github.io/rustup/overrides.html#the-toolchain-file
>
> On Thu, Aug 12, 2021 at 2:14 AM QP Hou <houqp....@gmail.com> wrote:
>
> > Hi Ruihang,
> >
> > Thanks for helping with the validation. It would certainly be helpful
> > if you could share the error log with me.
> >
> > I have also prepared an updated version of the verification script at
> >
> > https://github.com/houqp/arrow-datafusion/blob/qp_release/dev/release/verify-release-candidate.sh
> > .
> > This script does a clean checkout of everything before running tests
> > and linting tools. Could you give that a try to see if you are getting
> > the same results?
> >
> > Thanks,
> > QP
> >
> > On Wed, Aug 11, 2021 at 6:38 AM Wayne Xia <waynest...@gmail.com> wrote:
> > >
> > > Thanks, QP!
> > >
> > > I verified the signature and checked shasum, but got 3 failed case while
> > > testing:
> > >
> > > - execution_plans::shuffle_writer::tests::test
> > > - execution_plans::shuffle_writer::tests::test_partitioned
> > > -
> > physical_plan::repartition::tests::repartition_with_dropping_output_stream
> > >
> > > I set up env `ARROW_TEST_DATA` and `PARQUET_TEST_DATA`, then run the test
> > > with
> > > "cargo test --all --no-fail-fast" on Linux 5.13.6 with x86_64 chip.
> > >
> > > Did I miss something? I can paste the log here or file an issue if
> > needed.
> > >
> > > Ruihang
> > >
> > > QP Hou <houqp....@gmail.com>:
> > >
> > > > Hi,
> > > >
> > > > I would like to propose a release of Apache Arrow Datafusion
> > > > Implementation,
> > > > version 5.0.0.
> > > >
> > > > RC3 fixed a cargo publish issue discovered in RC1.
> > > >
> > > > This release candidate is based on commit:
> > > > deb929369c9aaba728ae0c2c49dcd05bfecc8bf8 [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. The vote will be open for at least 72 hours.
> > > >
> > > > [ ] +1 Release this as Apache Arrow Datafusion 5.0.0
> > > > [ ] +0
> > > > [ ] -1 Do not release this as Apache Arrow Datafusion 5.0.0 because...
> > > >
> > > > [1]:
> > > >
> > https://github.com/apache/arrow-datafusion/tree/deb929369c9aaba728ae0c2c49dcd05bfecc8bf8
> > > > [2]:
> > > >
> > https://dist.apache.org/repos/dist/dev/arrow/apache-arrow-datafusion-5.0.0-rc3
> > > > [3]:
> > > >
> > https://github.com/apache/arrow-datafusion/blob/deb929369c9aaba728ae0c2c49dcd05bfecc8bf8/CHANGELOG.md
> > > >
> > > > Thanks,
> > > > QP
> > > >
> >

Reply via email to