Skip to content

Conversation

cnauroth
Copy link
Contributor

Description of PR

Update license information for hadoop-gcp dependencies.

How was this patch tested?

mvn clean package -Pdist -Dtar -DskipTests

I confirmed the new license information was present in the built distro.

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

@github-actions github-actions bot added the trunk label Aug 22, 2025
@cnauroth
Copy link
Contributor Author

@arunkumarchacko FYI

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 22s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HADOOP-19343 Compile Tests _
+1 💚 shadedclient 31m 14s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shadedclient 23m 39s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 28s The patch does not generate ASF License warnings.
57m 13s
Subsystem Report/Notes
Docker ClientAPI=1.51 ServerAPI=1.51 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7891/1/artifact/out/Dockerfile
GITHUB PR #7891
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs
uname Linux 13e467fb644d 5.15.0-143-generic #153-Ubuntu SMP Fri Jun 13 19:10:45 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision HADOOP-19343 / 6cc0025
Max. process+thread count 546 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7891/1/console
versions git=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@slfan1989
Copy link
Contributor

@cnauroth The file LICENSE-auto-value-annotations.txt is empty. Do we need to submit this file?

@cnauroth cnauroth requested a review from slfan1989 August 25, 2025 15:35
@cnauroth cnauroth force-pushed the HADOOP-19343-licenses branch from 6cc0025 to fc426a0 Compare August 25, 2025 18:14
@cnauroth
Copy link
Contributor Author

@cnauroth The file LICENSE-auto-value-annotations.txt is empty. Do we need to submit this file?

Good catch! This one turned out to be Apache-licensed, so there was no need for a separate license file. I updated the PR and will merge later today.

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 8m 45s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HADOOP-19343 Compile Tests _
+1 💚 shadedclient 26m 22s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shadedclient 20m 12s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 27s The patch does not generate ASF License warnings.
57m 17s
Subsystem Report/Notes
Docker ClientAPI=1.51 ServerAPI=1.51 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7891/2/artifact/out/Dockerfile
GITHUB PR #7891
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs
uname Linux 39527193e313 5.15.0-143-generic #153-Ubuntu SMP Fri Jun 13 19:10:45 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision HADOOP-19343 / fc426a0
Max. process+thread count 546 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7891/2/console
versions git=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

cnauroth added a commit that referenced this pull request Aug 25, 2025
@cnauroth
Copy link
Contributor Author

@slfan1989 , thank you for the review. I committed this to the feature branch.

@cnauroth cnauroth closed this Aug 25, 2025
cnauroth added a commit that referenced this pull request Aug 25, 2025
cnauroth added a commit to cnauroth/hadoop that referenced this pull request Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants