Skip to content

follow up LICENSE/NOTICE fixes from the 1.11.0 RC4 thread #16396

@kevinjqliu

Description

@kevinjqliu

context: https://lists.apache.org/thread/ddw8cvyrvypoolmzhd673s5wglhdpd5f

-- LICENSE and NOTICE are good (nit: Presto and Delta Lake has NOTICE files that could be included in the Iceberg NOTICE)

  • AWS Bundle
    -- nit: jar contains LICENSE/NOTICE in the root, but also META-INF/LICENSE
    and META-INF/NOTICE. Usually, we should have only META-INF/LICENSE and
    META-INF/NOTICE. Also, it seems LICENSE and NOTICE are packaged twice in
    the jar.
    -- LICENSE and NOTICE are correct (nit: aws-crt-java, aws-eventstream-java
    have different NOTICE then the "regular" AWS Java SDK).
  • Azure Bundle
    -- LICENSE and NOTICE are good (nit: NOTICE could include NOTICE content
    from reactor-core and reactor-netty). As for AWS bundle, filtering
    META-INF/LICENSE and META-INF/NOTICE in the shade plugin would be good.
  • GCP Bundle
    -- LICENSE and NOTICE are good (as for AWS and Azure bundle, filtering on
    the shade plugin would be welcome).
  • Flink Runtime
    -- LICENSE and NOTICE are good. Nit: LICENSE and NOTICE are packaged twice
    in the jar file. META-INF/LICENSE* and META-INF/NOTICE* should be cleanup
    in favor of the Iceberg LICENSE/NOTICE.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions