I forgot to send this, left it in drafts :D

+1 - Ran test suite for Polaris on
https://github.com/apache/polaris/pull/442 - Ran all Iceberg Tests -
Checked Signatures and Sums - Confirmed Jar is signed by a loving father
and Iceberg PMC member

On Thu, Dec 5, 2024 at 7:31 PM Bryan Keller <brya...@gmail.com> wrote:

> Thanks to everyone who participated in the vote for Release Apache Iceberg
> 1.7.1 RC1.
>
> The vote result is:
>
> +1: 5 (binding), 2 (non-binding)
> +0: 0 (binding), 0 (non-binding)
> -1: 0 (binding), 0 (non-binding)
>
> Therefore, the release candidate is passed.
>
> -Bryan
>
>
> On Dec 3, 2024, at 8:48 PM, Honah J. <hon...@apache.org> wrote:
>
> +1 (binding)
>
> Bryan, thanks for running the release!
>
> Verified signature/checksum/license. Built against JDK17.
>
> Kevin, thanks for the update and explanation!
>
> Best regards,
> Honah
>
> On Tue, Dec 3, 2024 at 7:35 PM Kevin Liu <kevinjq...@apache.org> wrote:
>
>> Hey everyone,
>>
>> I wanted to follow up on the issue I encountered with the failed tests
>> for `:iceberg-kafka-connect:iceberg-kafka-connect-runtime:integrationTest`
>> using Java 21. The issue was caused by a port conflict with the test
>> container for Kafka Connect [1].
>>
>> After shutting down the conflicting port, I was able to successfully
>> build and run the tests using Java 21.
>>
>> Best,
>> Kevin Liu
>>
>> [1]
>> https://github.com/apache/iceberg/blob/6501d29b2d46c8d57f46ad646e2daf7d8865f646/kafka-connect/kafka-connect-runtime/docker/docker-compose.yml#L29-L49
>>
>>
>> On Tue, Dec 3, 2024 at 4:31 PM Yufei Gu <flyrain...@gmail.com> wrote:
>>
>>> +1(binding)
>>>
>>> Verified signature, checksum, and license check. Build passed.
>>> Apache Polaris Test suites passed with the rc1.
>>>
>>> Yufei
>>>
>>>
>>> On Tue, Dec 3, 2024 at 3:58 PM Kevin Liu <kevinjq...@apache.org> wrote:
>>>
>>>> +1 (non-binding)
>>>> Thanks for running the release!
>>>>
>>>> Verified signature, checksum, and license check. Built and tested using
>>>> JDK 17 (`17.0.6-zulu`)
>>>>
>>>>
>>>> I also tried building with Java 21 (`21.0.4-amzn`) but ran into the
>>>> following failed tests. I don't think this is blocking the current RC since
>>>> support for building with Java 21 was added recently to v1.7.0
>>>> <https://iceberg.apache.org/releases/#170-release> (issue #10474
>>>> <https://github.com/apache/iceberg/pull/10474>)
>>>> ```
>>>> > Task
>>>> :iceberg-kafka-connect:iceberg-kafka-connect-runtime:integrationTest
>>>>
>>>> IntegrationDynamicTableTest > initializationError FAILED
>>>>     org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:351
>>>>         Caused by: org.rnorth.ducttape.RetryCountExceededException at
>>>> Unreliables.java:88
>>>>             Caused by:
>>>> org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:555
>>>>                 Caused by: java.lang.IllegalStateException at
>>>> GenericContainer.java:487
>>>>
>>>> IntegrationMultiTableTest > initializationError FAILED
>>>>     org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:351
>>>>         Caused by: org.rnorth.ducttape.RetryCountExceededException at
>>>> Unreliables.java:88
>>>>             Caused by:
>>>> org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:555
>>>>                 Caused by: java.lang.IllegalStateException at
>>>> GenericContainer.java:487
>>>>
>>>> IntegrationTest > initializationError FAILED
>>>>     org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:351
>>>>         Caused by: org.rnorth.ducttape.RetryCountExceededException at
>>>> Unreliables.java:88
>>>>             Caused by:
>>>> org.testcontainers.containers.ContainerLaunchException at
>>>> GenericContainer.java:555
>>>>                 Caused by: java.lang.IllegalStateException at
>>>> GenericContainer.java:487
>>>>
>>>> 3 tests completed, 3 failed
>>>>
>>>> > Task
>>>> :iceberg-kafka-connect:iceberg-kafka-connect-runtime:integrationTest FAILED
>>>> ```
>>>>
>>>> The test report shows that all 3 tests failed on the
>>>> `initializationError` step with
>>>> ```
>>>> org.testcontainers.containers.ContainerLaunchException: Container
>>>> startup failed for image docker:24.0.2
>>>> ```
>>>>
>>>> and I double-checked that `docker` is indeed running on my laptop.
>>>>
>>>>
>>>> Best,
>>>> Kevin Liu
>>>>
>>>> On Tue, Dec 3, 2024 at 7:31 AM Jean-Baptiste Onofré <j...@nanthrax.net>
>>>> wrote:
>>>>
>>>>> +1 (non binding)
>>>>>
>>>>> Regards
>>>>> JB
>>>>>
>>>>> On Thu, Nov 21, 2024 at 2:35 PM Bryan Keller <brya...@gmail.com>
>>>>> wrote:
>>>>> >
>>>>> > Hi Everyone,
>>>>> >
>>>>> > I propose that we release the following RC as the official Apache
>>>>> Iceberg 1.7.1 release.
>>>>> >
>>>>> > The commit ID is 4a432839233f2343a9eae8255532f911f06358ef
>>>>> > * This corresponds to the tag: apache-iceberg-1.7.1-rc1
>>>>> > * https://github.com/apache/iceberg/commits/apache-iceberg-1.7.1-rc1
>>>>> > *
>>>>> https://github.com/apache/iceberg/tree/4a432839233f2343a9eae8255532f911f06358ef
>>>>> >
>>>>> > The release tarball, signature, and checksums are here:
>>>>> > *
>>>>> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.7.1-rc1
>>>>> >
>>>>> > 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-1178
>>>>> >
>>>>> > Please download, verify, and test.
>>>>> >
>>>>> > Please vote in the next 72 hours.
>>>>> >
>>>>> > [ ] +1 Release this as Apache Iceberg 1.7.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.
>>>>> >
>>>>> > (NOTE: The vote on 1.7.1 RC0 was skipped as a last minute bug fix
>>>>> came in.)
>>>>> >
>>>>>
>>>>
>

Reply via email to