> Xuanwo, if you want to learn more about voting, there is also an Apache page
> on it
> <https://www.apache.org/foundation/voting.html#expressing-votes-1-0-1-and-fractions>
> (that includes some suggestions :). But also feel welcome to ask on the
> devlist here.
Thanks for pointing out the docs address. Just learnt that vote is [-1, 1]
instead of {-1, 0, 1}.
On Thu, Oct 19, 2023, at 17:30, Fokko Driesprong wrote:
> Thanks Eduard for running this release!
>
> +1 (binding):
> • Checked the sha/signature
> • Ran our example notebooks against 1.4.1
> <https://github.com/tabular-io/docker-spark-iceberg/pull/109> and it looks
> well
> Xuanwo, if you want to learn more about voting, there is also an Apache page
> on it
> <https://www.apache.org/foundation/voting.html#expressing-votes-1-0-1-and-fractions>
> (that includes some suggestions :). But also feel welcome to ask on the
> devlist here.
>
> Kind regards,
> Fokko
>
>
> Op do 19 okt 2023 om 11:02 schreef Xuanwo <xua...@apache.org>:
>> __
>>> That said, from a community standpoint, it's good to take any -1 (binding
>>> or non binding) into account.
>>>
>>> In your case, I would have voted -0 (to avoid confusion).
>>
>> Lesson learned. Next time, if the same situation occurs, I'll vote -0 to
>> make my statement more clear.
>>
>> On Thu, Oct 19, 2023, at 16:23, Jean-Baptiste Onofré wrote:
>>> By the way, at Apache, it's not really possible to veto or block a release:
>>> you need three binding votes, even if we have a fourth binding vote with
>>> -1, the release can pass.
>>> That said, from a community standpoint, it's good to take any -1 (binding
>>> or non binding) into account.
>>>
>>> In your case, I would have voted -0 (to avoid confusion).
>>>
>>> You can see that I voted +1 because:
>>> - the release is the same as the previous ones
>>> - the issues have been identified and so we can fix it
>>>
>>> Regards
>>> JB
>>>
>>> On Thu, Oct 19, 2023 at 10:15 AM Xuanwo <xua...@apache.org> wrote:
>>>> __
>>>>> You can see it’s what I mentioned in my vote email. However, as it’s like
>>>>> this for a while, I voted +1 and I have PRs ready to be submitted
>>>>> (including rat execution).
>>>>>
>>>>> So do you think it’s blocking ?
>>>>
>>>> Thanks for the clarification.
>>>>
>>>> I'm voting -1 due to the reasons mentioned, but it doesn't block this
>>>> release (especially since it's non-binding). This release can proceed once
>>>> it garners enough +1 votes. My -1 vote is simply to highlight areas we
>>>> could improve in future releases.
>>>>
>>>>
>>>> On Thu, Oct 19, 2023, at 13:11, Jean-Baptiste Onofré wrote:
>>>>> Hi
>>>>>
>>>>> You can see it’s what I mentioned in my vote email. However, as it’s like
>>>>> this for a while, I voted +1 and I have PRs ready to be submitted
>>>>> (including rat execution).
>>>>>
>>>>> So do you think it’s blocking ?
>>>>>
>>>>> Regards
>>>>> JB
>>>>>
>>>>> Le mer. 18 oct. 2023 à 16:27, Xuanwo <xua...@apache.org> a écrit :
>>>>>> __
>>>>>> -1 (non-binding)
>>>>>>
>>>>>> - checksum and signature is good
>>>>>>
>>>>>> - the following files not have license
>>>>>> - .baseline/idea/intellij-java-palantir-style.xml
>>>>>> - .baseline/checkstyle/checkstyle.xml
>>>>>> - gradle/libs.versions.toml
>>>>>> - .baseline/checkstyle/checkstyle-suppressions.xml
>>>>>> - .baseline/checkstyle/checkstyle-suppressions.xml
>>>>>>
>>>>>> - release contains binary files
>>>>>> -
>>>>>> core/src/test/resources/org/apache/iceberg/puffin/v1/empty-puffin-uncompressed.bin
>>>>>> -
>>>>>> core/src/test/resources/org/apache/iceberg/puffin/v1/sample-metric-data-compressed-zstd.bin
>>>>>> -
>>>>>> core/src/test/resources/org/apache/iceberg/puffin/v1/sample-metric-data-uncompressed.bin
>>>>>>
>>>>>> On Wed, Oct 18, 2023, at 21:55, Eduard Tudenhoefner wrote:
>>>>>>> +1 (non-binding)
>>>>>>>
>>>>>>> * validated checksum and signature
>>>>>>> * checked license docs & ran RAT checks
>>>>>>> * ran build and tests with JDK8
>>>>>>> * ran into one test failure, which is reported in
>>>>>>> https://github.com/apache/iceberg/issues/8824, but this shouldn't block
>>>>>>> the release
>>>>>>> * tested with Trino in https://github.com/trinodb/trino/pull/19434
>>>>>>>
>>>>>>> On Wed, Oct 18, 2023 at 3:15 PM Jean-Baptiste Onofré
>>>>>>> <j...@nanthrax.net> wrote:
>>>>>>>> +1 (non binding)
>>>>>>>>
>>>>>>>> I checked:
>>>>>>>> * hashes and signatures are OK
>>>>>>>> * I did quick tests using spark 3.5
>>>>>>>>
>>>>>>>> I found the following issues that we should fix:
>>>>>>>> * the source distribution contains two binary files (used for tests,
>>>>>>>> empty-puffin-uncompressed.bin and
>>>>>>>> sample-metric-data-uncompressed.bin). Binary files should not be
>>>>>>>> included in the source distribution.
>>>>>>>> * some files don't contain ASF header
>>>>>>>>
>>>>>>>> I will work to fix these issues, and also, I will propose to include
>>>>>>>> rat to test our distribution.
>>>>>>>>
>>>>>>>> Regards
>>>>>>>> JB
>>>>>>>>
>>>>>>>>
>>>>>>>> On Wed, Oct 18, 2023 at 11:15 AM Eduard Tudenhoefner
>>>>>>>> <edu...@tabular.io> wrote:
>>>>>>>>> Hi Everyone,
>>>>>>>>>
>>>>>>>>> I propose that we release the following RC as the official Apache
>>>>>>>>> Iceberg 1.4.1 release.
>>>>>>>>>
>>>>>>>>> The commit ID is 445664fb8d82950215872cbfec91e37c5fa0920f
>>>>>>>>> * This corresponds to the tag: apache-iceberg-1.4.1-rc0
>>>>>>>>> * https://github.com/apache/iceberg/commits/apache-iceberg-1.4.1-rc0
>>>>>>>>> *
>>>>>>>>> https://github.com/apache/iceberg/tree/445664fb8d82950215872cbfec91e37c5fa0920f
>>>>>>>>>
>>>>>>>>> The release tarball, signature, and checksums are here:
>>>>>>>>> *
>>>>>>>>> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.4.1-rc0
>>>>>>>>>
>>>>>>>>> You can find the KEYS file here:
>>>>>>>>> * https://dist.apache.org/repos/dist/dev/iceberg/KEYS
>>>>>>>>>
>>>>>>>>> Convenience binary artifacts are staged on Nexus. The Maven
>>>>>>>>> repository URL is:
>>>>>>>>> *
>>>>>>>>> https://repository.apache.org/content/repositories/orgapacheiceberg-1147/
>>>>>>>>>
>>>>>>>>> Please download, verify, and test.
>>>>>>>>>
>>>>>>>>> Please vote in the next 72 hours.
>>>>>>>>>
>>>>>>>>> [ ] +1 Release this as Apache Iceberg 1.4.1
>>>>>>>>> [ ] +0
>>>>>>>>> [ ] -1 Do not release this because...
>>>>>>>>>
>>>>>>>>> Only PMC members have binding votes, but other community members are
>>>>>>>>> encouraged to cast
>>>>>>>>> non-binding votes. This vote will pass if there are 3 binding +1
>>>>>>>>> votes and more binding
>>>>>>>>> +1 votes than -1 votes.
>>>>>>>>>
>>>>>>>>>
>>>>>>
>>>>>> Xuanwo
>>>>>>
>>>>
>>>> Xuanwo
>> Xuanwo
>>
Xuanwo
