+1 (binding)

I have tested both:

TEST_DEFAULT=0 TEST_SOURCE=1 TEST_BINARIES=0
dev/release/verify-release-candidate.sh 17.0.0 2
and
TEST_DEFAULT=0 TEST_SOURCE=0 TEST_BINARIES=1
dev/release/verify-release-candidate.sh 17.0.0 2

On:
  * Ubuntu 22.04 LTS
  * Python 3.10.12
  * gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
  * NVIDIA CUDA cuda_11.5.r11.5/compiler.30672275_0
  * openjdk 17.0.11 2024-04-16
  * ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x86_64-linux-gnu]
  * dotnet 8.0.107

Raúl

El lun, 15 jul 2024 a las 15:55, Neal Richardson
(<neal.p.richard...@gmail.com>) escribió:
>
> Adam,
> Yes, I saw that when it was introduced here, on a PR I did:
> https://github.com/apache/arrow/runs/25894658283
>
> It's on the smallest scale factor and on two of the queries (
> https://github.com/voltrondata-labs/arrowbench/blob/main/R/tpch-queries.R#L510-L535
> and
> https://github.com/voltrondata-labs/arrowbench/blob/main/R/tpch-queries.R#L583-L599).
> I looked into it, and after digging into the limited output available from
> conbench, I wasn't convinced that it was meaningful. My theory based on the
> changes I made is that the queries themselves aren't actually running in
> Acero, that for some reason they're failing to translate into arrow
> expressions and are falling back to running in regular dplyr in R. I made a
> change in that PR that would make that fallback code path slower in some
> circumstances. So the benchmarks may have regressed, but they aren't really
> testing Arrow code--at least that's my theory.
>
> Of course, "no regression" would be preferable. I didn't (and don't now)
> have time to dig further into it, but others are welcome to if they can.
>
> Neal
>
>
>
> On Mon, Jul 15, 2024 at 9:07 AM Adam Lippai <a...@rigo.sk> wrote:
>
> > Do I read it correctly that there is tpch regression in the R benchmark?
> >
> > Best regards,
> > Adam Lippai
> >
> > On Mon, Jul 15, 2024 at 06:05 Fokko Driesprong <fo...@apache.org> wrote:
> >
> > > Thanks to everyone who contributed to the new release!
> > >
> > > +1 (non-binding)
> > >
> > > I've tested against PyIceberg
> > > <https://github.com/apache/iceberg-python/pull/929>.
> > >
> > > Kind regards,
> > > Fokko
> > >
> > > Op ma 15 jul 2024 om 10:29 schreef David Li <lidav...@apache.org>:
> > >
> > > > +1 (binding)
> > > >
> > > > Tested on Debian 12/x86_64
> > > >
> > > > On Mon, Jul 15, 2024, at 15:31, Gang Wu wrote:
> > > > > +1 (non-binding)
> > > > >
> > > > > Verified C++ on my M1 Mac by running:
> > > > > - TEST_DEFAULT=0 TEST_CPP=1 ./verify-release-candidate.sh 17.0.0 2
> > > > >
> > > > > BTW, I ran into this issue as well:
> > > > > https://github.com/apache/arrow/issues/43167
> > > > >
> > > > > Best,
> > > > > Gang
> > > > >
> > > > > On Mon, Jul 15, 2024 at 1:39 PM Jean-Baptiste Onofré <
> > j...@nanthrax.net>
> > > > > wrote:
> > > > >
> > > > >> +1 (non binding)
> > > > >>
> > > > >> Regards
> > > > >> JB
> > > > >>
> > > > >> On Fri, Jul 12, 2024 at 11:56 AM Raúl Cumplido <rau...@apache.org>
> > > > wrote:
> > > > >> >
> > > > >> > Hi,
> > > > >> >
> > > > >> > I would like to propose the following release candidate (RC2) of
> > > > Apache
> > > > >> > Arrow version 17.0.0. This is a release consisting of 321
> > > > >> > resolved GitHub issues[1].
> > > > >> >
> > > > >> > This release candidate is based on commit:
> > > > >> > 6a2e19a852b367c72d7b12da4d104456491ed8b7 [2]
> > > > >> >
> > > > >> > The source release rc2 is hosted at [3].
> > > > >> > The binary artifacts are hosted at [4][5][6][7][8][9][10][11].
> > > > >> > The changelog is located at [12].
> > > > >> >
> > > > >> > Please download, verify checksums and signatures, run the unit
> > > tests,
> > > > >> > and vote on the release. See [13] for how to validate a release
> > > > >> candidate.
> > > > >> >
> > > > >> > See also a verification result on GitHub pull request [14].
> > > > >> >
> > > > >> > The vote will be open for at least 72 hours.
> > > > >> >
> > > > >> > [ ] +1 Release this as Apache Arrow 17.0.0
> > > > >> > [ ] +0
> > > > >> > [ ] -1 Do not release this as Apache Arrow 17.0.0 because...
> > > > >> >
> > > > >> > [1]:
> > > > >>
> > > >
> > >
> > https://github.com/apache/arrow/issues?q=is%3Aissue+milestone%3A17.0.0+is%3Aclosed
> > > > >> > [2]:
> > > > >>
> > > >
> > >
> > https://github.com/apache/arrow/tree/6a2e19a852b367c72d7b12da4d104456491ed8b7
> > > > >> > [3]:
> > > > >>
> > https://dist.apache.org/repos/dist/dev/arrow/apache-arrow-17.0.0-rc2
> > > > >> > [4]: https://apache.jfrog.io/artifactory/arrow/almalinux-rc/
> > > > >> > [5]: https://apache.jfrog.io/artifactory/arrow/amazon-linux-rc/
> > > > >> > [6]: https://apache.jfrog.io/artifactory/arrow/centos-rc/
> > > > >> > [7]: https://apache.jfrog.io/artifactory/arrow/debian-rc/
> > > > >> > [8]: https://apache.jfrog.io/artifactory/arrow/java-rc/17.0.0-rc2
> > > > >> > [9]:
> > https://apache.jfrog.io/artifactory/arrow/nuget-rc/17.0.0-rc2
> > > > >> > [10]:
> > > https://apache.jfrog.io/artifactory/arrow/python-rc/17.0.0-rc2
> > > > >> > [11]: https://apache.jfrog.io/artifactory/arrow/ubuntu-rc/
> > > > >> > [12]:
> > > > >>
> > > >
> > >
> > https://github.com/apache/arrow/blob/6a2e19a852b367c72d7b12da4d104456491ed8b7/CHANGELOG.md
> > > > >> > [13]:
> > > > https://arrow.apache.org/docs/developers/release_verification.html
> > > > >> > [14]: https://github.com/apache/arrow/pull/43220
> > > > >>
> > > >
> > >
> >

Reply via email to