Skip to content

Comments

Fix license and notice issue#2702

Closed
luoyuxia wants to merge 3 commits intoapache:mainfrom
luoyuxia:fix-license-issue
Closed

Fix license and notice issue#2702
luoyuxia wants to merge 3 commits intoapache:mainfrom
luoyuxia:fix-license-issue

Conversation

@luoyuxia
Copy link
Contributor

@luoyuxia luoyuxia commented Feb 21, 2026

Purpose

Linked issue: close #2697

  • LICENSE should be a bit updated. For instance, mvnw is not part/used in
    the code of the distribution.

  • The licenses folder contains some confusing
    files (for instance LICENSE-re2j and LICENSE.re2j, which one is relevant,
    LICENSE.cddlv1.1 applies to which dependency ?, ...)

  • The NOTICE is not
    fully correct: the NOTICE should include the NOTICE for Apache
    dependencies. The list of BSD licensed dependencies (like jsch, paranamer,
    ...) should not be in the NOTICE, it should be in LICENSE.

Brief change log

1: remove mvnw from binary release
2: unify LICENSE-re2j and LICENSE.re2j
3: add a LICENSE file to list BSD(Non-Apache) licensed dependencies, remove the listed from NOTICE
After go through https://infra.apache.org/licensing-howto.html#permissive-deps, seem it recommand to keep NOTICE as simple as possible.

Tests

API and Format

Documentation

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Fluss’ release/legal artifacts to address ASF licensing/notice compliance (issue #2697), including generating binary distribution LICENSE/NOTICE variants and moving non-Apache dependency listings out of NOTICE into LICENSE files.

Changes:

  • Add release helper scripts to derive binary LICENSE/NOTICE by removing the Maven Wrapper section.
  • Update create_binary_release.sh to emit derived LICENSE/NOTICE into the binary tarball.
  • Refresh multiple module META-INF/NOTICE files to remove non-Apache dependency listings, and introduce corresponding META-INF/LICENSE (+ license texts under META-INF/licenses/) for bundled third-party components.

Reviewed changes

Copilot reviewed 24 out of 28 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/releasing/derive_binary_notice.sh New script to generate binary NOTICE without Maven Wrapper notice block.
tools/releasing/derive_binary_license.sh New script to generate binary LICENSE without mvnw references.
tools/releasing/create_binary_release.sh Use derive scripts instead of copying root LICENSE/NOTICE into the binary bundle.
fluss-server/src/main/resources/META-INF/NOTICE Remove MIT/BSD dependency listing section from NOTICE.
fluss-server/src/main/resources/META-INF/LICENSE Add module LICENSE containing Apache License text + third-party license summary.
fluss-lake/fluss-lake-lance/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-lake/fluss-lake-lance/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-lake/fluss-lake-iceberg/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-s3/src/main/resources/META-INF/licenses/LICENSE.stax2api Add third-party license text for stax2-api.
fluss-filesystems/fluss-fs-s3/src/main/resources/META-INF/licenses/LICENSE.re2j Add third-party license text for re2j.
fluss-filesystems/fluss-fs-s3/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-s3/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-oss/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-oss/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-obs/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-obs/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-hdfs/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-hdfs/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-hadoop-shaded/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-hadoop-shaded/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-gs/src/main/resources/META-INF/licenses/LICENSE.stax2api Add third-party license text for stax2-api.
fluss-filesystems/fluss-fs-gs/src/main/resources/META-INF/licenses/LICENSE.re2j Add third-party license text for re2j.
fluss-filesystems/fluss-fs-gs/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-gs/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-filesystems/fluss-fs-azure/src/main/resources/META-INF/NOTICE Remove non-Apache dependency listing sections from NOTICE.
fluss-filesystems/fluss-fs-azure/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.
fluss-client/src/main/resources/META-INF/NOTICE Remove MIT/BSD dependency listing sections from NOTICE.
fluss-client/src/main/resources/META-INF/LICENSE Add module LICENSE with Apache License text + third-party license summary.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@luoyuxia luoyuxia changed the title Fix license issue Fix license and notice issue Feb 21, 2026
@jbonofre
Copy link
Member

Proposed complete fix: #2703

@luoyuxia
Copy link
Contributor Author

@jbonofre Thank you very much!!! I'll close mine and revew yours.

@luoyuxia luoyuxia closed this Feb 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix license and notice issue

2 participants