I'm working on the licensing fix.

Regards
JB

On Thu, Feb 20, 2025 at 3:03 PM Eduard Tudenhöfner
<etudenhoef...@apache.org> wrote:
>
> @Yuya you're absolutely right and thanks for reporting this. This was a 
> mistake on my part that I didn't catch. I've created 
> https://github.com/apache/iceberg/pull/12351 to fix this.
>
> I'm cancelling this RC and will create RC1 once that PR and the licensing 
> stuff has been fixed.
>
>
>
> On Thu, Feb 20, 2025 at 11:22 AM Yuya Ebihara 
> <yuya.ebih...@starburstdata.com> wrote:
>>
>> Hi,
>>
>> Thank you for preparing RC, Eduard.
>>
>> I tested 1.8.1 on Trino. The S3-compatible storage issue has been resolved, 
>> but Unity catalog and tabulario/iceberg-rest:1.6.0 tests are still failing.
>> Those catalogs don't return endpoints in /v1/config, so they fall back to 
>> DEFAULT_ENDPOINTS.
>> Thus, RESTSessionCatalog internally calls unsupported HEAD endpoints and 
>> requests to check the existence of namespace/table/view fail.
>>
>> BR,
>> Yuya
>>
>> On Thu, Feb 20, 2025 at 7:00 PM Jean-Baptiste Onofré <j...@nanthrax.net> 
>> wrote:
>>>
>>> +0 (non binding)
>>>
>>> I checked:
>>> - hash and checksum are good
>>> - LICENSE and NOTICE are good in the source distribution
>>> - no binary file found in the source distribution
>>> - ASF header is present in all expected file
>>> - able to build from the source distribution
>>> - I did "smoke tests" with Spark and Polaris without problem
>>> - the LICENSE is iceberg-aws-bundle is not correct:
>>>    - netty-buffer version mentioned in LICENSE is 4.1.112.Final, but
>>> the version actually shaded in jar is 4.1.115.Final (but it's correct
>>> in NOTICE)
>>>    - aws-crt version mentioned in LICENSE is 0.33.6, but the version
>>> actually shaded in jar is 0.33.3 (and it's not present in NOTICE)
>>> - the LICENSE in iceberg-azure-bundle is not correct:
>>>    - azure-identity version mentioned in LICENSE is 1.14.2, but the
>>> version actually shaded in jar is 1.15.0
>>> - the LICENSE in iceberg-gcp-bundle is not correct:
>>>    - grpc-google-cloud-storage-v2 version mentioned in LICENSE is
>>> 2.22.5-alpha, but the version actually shaded in jar is 2.47.0
>>>    - proto-google-cloud-storage-v2 version mentioned in LICENSE is
>>> 2.22.5-alpha, but the version actually shaded in jar is 2.47.0
>>>  - the LICENSE in kafka-connect-runtime main/hive distributions is not 
>>> correct:
>>>    - opentelemetry artifacts are missing in LICENSE
>>>
>>> As said in a previous PR comment, when reviewing dependabot PRs, the
>>> reviewer should add a commit directly in the dependabot PR keeping
>>> LICENSE/NOTICE in sync. If it's too painful, I think it's acceptable
>>> to use "major" dependencies version in the LICENSE/NOTICE (like 1.5.x
>>> instead of 1.5.6).
>>>
>>> I will propose PRs to fix the versions in LICENSE/NOTICE and use
>>> "major" versions.
>>>
>>> Regards
>>> JB
>>>
>>> On Thu, Feb 20, 2025 at 9:41 AM Eduard Tudenhoefner
>>> <etudenhoef...@gmail.com> wrote:
>>> >
>>> > Hi Everyone,
>>> >
>>> > I propose that we release the following RC as the official Apache Iceberg 
>>> > 1.8.1 release.
>>> >
>>> > The commit ID is 673192d9152a5532f2a84aa2fc815cfa16f7f203
>>> > * This corresponds to the tag: apache-iceberg-1.8.1-rc0
>>> > * https://github.com/apache/iceberg/commits/apache-iceberg-1.8.1-rc0
>>> > * 
>>> > https://github.com/apache/iceberg/tree/673192d9152a5532f2a84aa2fc815cfa16f7f203
>>> >
>>> > The release tarball, signature, and checksums are here:
>>> > * https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.8.1-rc0
>>> >
>>> > You can find the KEYS file here:
>>> > * https://downloads.apache.org/iceberg/KEYS
>>> >
>>> > Convenience binary artifacts are staged on Nexus. The Maven repository 
>>> > URL is:
>>> > * 
>>> > https://repository.apache.org/content/repositories/orgapacheiceberg-1183/
>>> >
>>> > Please download, verify, and test.
>>> >
>>> > Please vote in the next 72 hours.
>>> >
>>> > [ ] +1 Release this as Apache Iceberg 1.8.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.
>>> >

Reply via email to