Thanks for the vote, and I'm aware that it is still the holiday season, so
no apologies are needed for the delay.

Kind regards,
Fokko Driesprong


Op vr 6 jan. 2023 om 19:02 schreef Jack Ye <yezhao...@gmail.com>:

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

Reply via email to