poorbarcode opened a new pull request, #21439:
URL: https://github.com/apache/pulsar/pull/21439

   ### Motivation
   
   The dependency's version of `animal-sniffer-annotations` is `1.21` now, but 
the license of presto(trino) is still `1.19`
   
   see: 
https://github.com/apache/pulsar/actions/runs/6627474211/job/18019187456?pr=21435
   ```
   Run src/check-binary-license.sh 
./distribution/server/target/apache-pulsar-*-bin.tar.gz && 
src/check-binary-license.sh 
./distribution/shell/target/apache-pulsar-shell-*-bin.tar.gz
     src/check-binary-license.sh 
./distribution/server/target/apache-pulsar-*-bin.tar.gz && 
src/check-binary-license.sh 
./distribution/shell/target/apache-pulsar-shell-*-bin.tar.gz
     shell: /usr/bin/bash -e {0}
     env:
       MAVEN_OPTS: -Xss1500k -Xmx1024m 
-Daether.connector.http.reuseConnections=false 
-Daether.connector.requestTimeout=60000 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard 
-Dmaven.wagon.http.retryHandler.count=3 
-Dmaven.wagon.http.retryHandler.requestSentEnabled=true 
-Dmaven.wagon.http.serviceUnavailableRetryStrategy.class=standard 
-Dmaven.wagon.rto=60000
       ARTIFACT_RETENTION_DAYS: 3
       GRADLE_ENTERPRISE_ACCESS_KEY: 
       JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.8-1/x64
       JAVA_HOME_17_X64: 
/opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.8-1/x64
       ACTIONS_RUNTIME_TOKEN: ***
       ACTIONS_RUNTIME_URL: 
https://pipelinesghubeus21.actions.githubusercontent.com/EXIZfX8ixSbQEfqDA00dMJrEtloTMSwjlUs9NqJqrTcJfd7NaR/
       UBUNTU_MIRROR: http://azure.archive.ubuntu.com/ubuntu/
       UBUNTU_SECURITY_MIRROR: http://azure.archive.ubuntu.com/ubuntu/
   src/check-binary-license.sh: line 69: echo: write error: Broken pipe
   src/check-binary-license.sh: line 69: echo: write error: Broken pipe
   animal-sniffer-annotations-1.21.jar unaccounted for in trino/LICENSE
   animal-sniffer-annotations-1.19.jar mentioned in trino/LICENSE, but not 
bundled
   
   It looks like there are issues with the LICENSE/NOTICE.
   Error: Process completed with exit code 2.
   ```
   ### Modifications
   
   Correct the license file
   
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: x


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to