+1 (binding)

Verified sigs/sums/license/build/test (Python 3.11.6)

All checks out,
-Dan

On Thu, Apr 4, 2024 at 4:39 PM Hussein Awala <huss...@awala.fr> wrote:

> +1 (non-binding)
>
> - Verified signatures, checksums, and license
> - Tested creating and reading a non-partitioned table with the Glue catalog
>
> On Fri, Apr 5, 2024 at 12:44 AM Justin Mclean <jus...@classsoftware.com>
> wrote:
>
>> HI,
>>
>> Thanks for that. I don't understand "we don't bundle the code, but just
>> took some part of it”. Either the code is in the source release or not in
>> the source release; if any part of it is in the source release, then it is
>> bundled. The LICENSE and NOTICE files need to relate to what is included in
>> the release artefact. Looking at both those PRs, it seems in both cases,
>> you do bundle code, but I understand why you might not want to mention th
>> autogenerated code. Either way, I think a little more is needed here to
>> make the NOTICE file in line with ASF policy.
>>
>> Kind Regards,
>> Justin
>
>

Reply via email to