MartijnVisser commented on PR #19428:
URL: https://github.com/apache/flink/pull/19428#issuecomment-1096319860

   Thanks for the PR @David-N-Perkins but we need to also make changes to the 
NOTICE files, since this newer version has different version numbers
   
   ```
   
   20:46:51,337 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.grpc:grpc-core:1.26.0 is mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-codec-http2:4.1.51.Final is mentioned in 
NOTICE file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was 
not mentioned by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-handler-proxy:4.1.51.Final is mentioned in 
NOTICE file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was 
not mentioned by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency net.jpountz.lz4:lz4:1.3.0 is mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.google.protobuf:protobuf-java:3.11.0 is mentioned in 
NOTICE file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was 
not mentioned by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-handler:4.1.51.Final is mentioned in NOTICE 
file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not 
mentioned by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.grpc:grpc-stub:1.26.0 is mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency org.bouncycastle:bcprov-jdk15on:1.54 is mentioned in NOTICE 
file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not 
mentioned by the build output as a bundled dependency
   20:46:51,338 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-codec-http:4.1.51.Final is mentioned in 
NOTICE file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was 
not mentioned by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.google.guava:guava:26.0-jre is mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.jcraft:jzlib:1.1.3 is mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.google.auth:google-auth-library-credentials:0.18.0 is 
mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-codec:4.1.51.Final is mentioned in NOTICE 
file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not 
mentioned by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-transport-native-unix-common:4.1.51.Final is 
mentioned in NOTICE file 
/__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not mentioned 
by the build output as a bundled dependency
   20:46:51,339 DEBUG org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency io.netty:netty-common:4.1.51.Final is mentioned in NOTICE 
file /__w/1/s/flink-python/src/main/resources/META-INF/NOTICE, but was not 
mentioned by the build output as a bundled dependency
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency com.facebook.presto:presto-hive:0.272 in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency org.apache.hudi:hudi-presto-bundle:0.10.1 in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency com.facebook.presto:presto-hive-common:0.272 
in NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency 
com.facebook.presto:presto-hive-metastore:0.272 in NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency org.alluxio:alluxio-shaded-client:2.7.3 in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 ERROR org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Could not find dependency com.facebook.presto:presto-common:0.272 in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE
   20:46:51,342 WARN  org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.facebook.presto:presto-common:0.257 is mentioned in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE,
 but is not expected there
   20:46:51,342 WARN  org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.facebook.presto:presto-hive-metastore:0.257 is mentioned 
in NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE,
 but is not expected there
   20:46:51,342 WARN  org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency org.alluxio:alluxio-shaded-client:2.5.0-3 is mentioned in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE,
 but is not expected there
   20:46:51,342 WARN  org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.facebook.presto:presto-hive:0.257 is mentioned in NOTICE 
file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE,
 but is not expected there
   20:46:51,342 WARN  org.apache.flink.tools.ci.licensecheck.NoticeFileChecker  
   [] - Dependency com.facebook.presto:presto-hive-common:0.257 is mentioned in 
NOTICE file 
/__w/1/s/flink-filesystems/flink-s3-fs-presto/src/main/resources/META-INF/NOTICE,
 but is not expected there
   20:46:51,466 INFO  org.apache.flink.tools.ci.licensecheck.JarFileChecker     
   [] - Checking directory /tmp/flink-validation-deployment with a total of 198 
jar files.
   20:48:28,002 WARN  org.apache.flink.tools.ci.licensecheck.LicenseChecker     
   [] - Found a total of 6 severe license issues
   
==============================================================================
   License Check failed. See previous output for details.
   
==============================================================================
   ```


-- 
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