Skip to content

Conversation

@Technoboy-
Copy link
Contributor

@Technoboy- Technoboy- commented Aug 24, 2022

Motivation

When run license check, it throw the following exception :

amqp-client-5.5.3.jar unaccounted for in lib/presto/LICENSE
commons-collections4-4.4.jar unaccounted for in lib/presto/LICENSE
jsr305-2.0.1.jar unaccounted for in lib/presto/LICENSE
log4j-api-2.18.0.jar unaccounted for in lib/presto/LICENSE
log4j-core-2.18.0.jar unaccounted for in lib/presto/LICENSE
log4j-slf4j-impl-2.18.0.jar unaccounted for in lib/presto/LICENSE
metrics-jmx-4.1.12.1.jar unaccounted for in lib/presto/LICENSE
netty-transport-native-epoll-4.1.77.Final.jar unaccounted for in lib/presto/LICENSE
rocksdbjni-6.29.4.1.jar unaccounted for in lib/presto/LICENSE
rxjava-3.0.1.jar unaccounted for in lib/presto/LICENSE
listenablefuture-9999.0-empty-to-avoid-conflict-with-guava.jar mentioned in lib/presto/LICENSE, but not bundled
failsafe-2.4.4.jar mentioned in lib/presto/LICENSE, but not bundled
jersey-entity-filtering-2.34.jar mentioned in lib/presto/LICENSE, but not bundled
jersey-media-json-jackson-2.34.jar mentioned in lib/presto/LICENSE, but not bundled
jersey-media-multipart-2.34.jar mentioned in lib/presto/LICENSE, but not bundled

Documentation

  • doc-not-needed
    (Please explain why)

@Technoboy- Technoboy- changed the title Fix license issue. [branch-2.11][license] Fix license issue. Aug 24, 2022
@Technoboy- Technoboy- self-assigned this Aug 24, 2022
@Technoboy- Technoboy- changed the title [branch-2.11][license] Fix license issue. [[Dont' merge]branch-2.11][license] Fix license issue. Aug 24, 2022
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Aug 24, 2022
- presto-record-decoder-334.jar
* RocksDB JNI
- rocksdbjni-6.10.2.jar
- rocksdbjni-6.29.4.1.jar
Copy link
Contributor Author

@Technoboy- Technoboy- Aug 25, 2022

Choose a reason for hiding this comment

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

Should update as it may miss by #15142

- commons-cli-1.5.0.jar
- commons-codec-1.15.jar
- commons-collections4-4.1.jar
- commons-collections4-4.4.jar
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should update

@tisonkun
Copy link
Member

We don't have a CI check guard for this regression. See #16783.

@Technoboy-
Copy link
Contributor Author

We don't have a CI check guard for this regression. See #16783.

We need to fix this.

@Technoboy- Technoboy- changed the title [[Dont' merge]branch-2.11][license] Fix license issue. [branch-2.11][license] Fix license issue. Aug 25, 2022
@Technoboy- Technoboy- merged commit 2e21b44 into apache:branch-2.11 Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs release/2.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants