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 >>> >>
