+1 (binding) Just came back from vacation and am trying to catch up, sorry for the delay of vote.
- Verified checksum, license, signature - Verified tests passing - Verified Glue catalog related operations working - was able to reproduce the 2 issues that are identified, but those are non blocking as discussed. Best, Jack Ye On Fri, Jan 6, 2023 at 9:46 AM Driesprong, Fokko <fo...@apache.org> wrote: > Hi Ryan, Daniel, > > Thanks for giving it a try and for the voting. > > Tests are passing, although in a new environment I get the error about >> not being able to load pyparsing. It would be nice to fix that, but since >> it isn't a regression, I think this release should not be blocked by it. >> > > This was fixed in #6439 <https://github.com/apache/iceberg/pull/6439> and > should probably have been backported to 0.2.1 as well. > > - tests fail if you have a ~/.pyiceberg.yaml configuration (we should >> probably fix this, but is not a blocker) > > > This was fixed in #6445 <https://github.com/apache/iceberg/pull/6445>, and > I've also created a PR <https://github.com/apache/iceberg/pull/6521> to > add the suggestion of using a fresh docker container. > > Is there one more PMC around for a vote? :) > > Cheers, Fokko > > > > Op di 3 jan. 2023 om 01:33 schreef Daniel Weeks <daniel.c.we...@gmail.com > >: > >> +1 (binding) >> >> Verified: >> - sigs/sums/license/tests >> - removal of hard dependency on pyarrow >> >> I don't believe the following should block this release unless they were >> introduced in this patch version, but I ran into the following: >> - pyparsing was not installed during 'make install' and tests failed >> - tests fail if you have a ~/.pyiceberg.yaml configuration (we should >> probably fix this, but is not a blocker) >> >> -Dan >> >> On Mon, Jan 2, 2023 at 12:22 AM Driesprong, Fokko <fo...@driesprong.frl> >> wrote: >> >>> Hey everyone! >>> >>> First of all, happy new year! Best wishes for 2023 to you and your >>> family. Just a gentle reminder to see if can release this RC to fix some >>> critical bugs in 0.2.0. >>> >>> Thanks! >>> >>> Fokko Driesprong >>> >>> Op di 27 dec. 2022 om 09:17 schreef Driesprong, Fokko <fo...@apache.org >>> >: >>> >>>> Hi Everyone, >>>> >>>> >>>> First of all, I hope y'all had a great Christmas! >>>> >>>> >>>> Last week we fixed <https://github.com/apache/iceberg/pull/6478> an >>>> issue <https://github.com/apache/iceberg/issues/6469> in PyIceberg >>>> that breaks filtering (and partitioning) on date fields. As discussed in >>>> the Iceberg sync, I propose that we release the following RC as the >>>> official PyIceberg 0.2.1 release. >>>> >>>> >>>> This adds the following commits on top of 0.2.0: >>>> >>>> >>>> - Python: Read date as an int #6487 >>>> <https://github.com/apache/iceberg/pull/6487> >>>> - Python: Bump version to 0.2.1 #6483 >>>> <https://github.com/apache/iceberg/pull/6483> >>>> - Python: Fix reading UUIDs #6486 >>>> <https://github.com/apache/iceberg/pull/6486> >>>> - Python: Fix PyArrow import #6484 >>>> <https://github.com/apache/iceberg/pull/6484> >>>> >>>> Instructions on how to verify the release can be found here >>>> <https://py.iceberg.apache.org/verify-release/>. >>>> >>>> >>>> The commit ID is d48b60e71232fe5c27ca4f4238adeefa1faa9ddf >>>> >>>> >>>> * This corresponds to the tag: pyiceberg-0.2.1rc0 >>>> (072c7b6806f13ea314e1293d71fc9243b6b4be50) >>>> >>>> * https://github.com/apache/iceberg/releases/tag/pyiceberg-0.2.1rc0 >>>> >>>> * >>>> https://github.com/apache/iceberg/tree/d48b60e71232fe5c27ca4f4238adeefa1faa9ddf >>>> >>>> >>>> The release tarball, signature, and checksums are here: >>>> >>>> >>>> * https://dist.apache.org/repos/dist/dev/iceberg/pyiceberg-0.2.1rc0/ >>>> >>>> >>>> You can find the KEYS file here: >>>> >>>> >>>> * https://dist.apache.org/repos/dist/dev/iceberg/KEYS >>>> >>>> >>>> Convenience binary artifacts are staged on pypi: >>>> >>>> >>>> https://pypi.org/project/pyiceberg/0.2.1rc0/ >>>> >>>> >>>> And can be installed using: pip3 install pyiceberg==0.2.1rc0 >>>> >>>> >>>> Please download, verify, and test. >>>> >>>> >>>> Please vote in the next 72 hours. >>>> >>>> [ ] +1 Release this as PyIceberg 0.2.1 >>>> >>>> [ ] +0 >>>> >>>> [ ] -1 Do not release this because... >>>> >>>> >>>> Kind regards, >>>> >>>> Fokko Driesprong >>>> >>>
