Thank you Wes, I will keep that in mind for the next release. I also noticed I was adding the result prefix incorrectly while I was going through older results threads yesterday. I have since added a calling votes section in our release procedure documentation to reflect this.
On Sun, Aug 15, 2021 at 1:46 AM Wes McKinney <wesmck...@gmail.com> wrote: > > A couple of nitpicks with the result thread: > > * I counted 4, not 3 binding votes > * the usual result subject line starts with “[RESULT][VOTE]” which makes > vote results easier to locate when searching. > > Thank you for being the release manager! > > On Sat, Aug 14, 2021 at 9:56 PM QP Hou <houqp....@gmail.com> wrote: > > > The vote has passed with three +1 votes. Thank you to all who helped > > with the release verification. > > > > We have published new versions of datafusion and ballista to crates.io as > > well: > > > > https://crates.io/crates/datafusion/5.0.0 > > https://crates.io/crates/ballista/0.5.0 > > https://crates.io/crates/ballista-core/0.5.0 > > https://crates.io/crates/ballista-executor/0.5.0 > > https://crates.io/crates/ballista-scheduler/0.5.0 > > > > We didn't release the python binding to PyPI this time because it > > requires a binary release, which was not included in this vote. We are > > working on improving the process to include the python binary in the > > release process. > > > > Thanks, > > QP > > > > On Fri, Aug 13, 2021 at 2:34 PM Neville Dipale <nevilled...@gmail.com> > > wrote: > > > > > > +1 (bniding) > > > > > > i verified the RC on aarch64-macos > > > > > > On Fri, 13 Aug 2021 at 23:29, Jorge Cardoso Leitão < > > jorgecarlei...@gmail.com> > > > wrote: > > > > > > > +1 > > > > > > > > Great work everyone! > > > > > > > > > > > > On Fri, Aug 13, 2021, 22:19 Daniël Heres <danielhe...@gmail.com> > > wrote: > > > > > > > > > +1 (non binding). Looking good. > > > > > > > > > > > > > > > On Fri, Aug 13, 2021, 07:49 QP Hou <houqp....@gmail.com> wrote: > > > > > > > > > > > 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 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
