[GitHub] [druid] clintropolis commented on a change in pull request #9129: Bump Apache Parquet to 1.11.0

[GitBox <gi...@apache.org>]

clintropolis commented on a change in pull request #9129: Bump Apache Parquet to 1.11.0
URL: https://github.com/apache/druid/pull/9129#discussion_r365643348
 
 

 ##########
 File path: licenses.yaml
 ##########
 @@ -3135,7 +3135,7 @@ name: Apache Parquet
 license_category: binary
 module: extensions/druid-parquet-extensions
 license_name: Apache License version 2.0
-version: 1.10.1
+version: 1.11.0
 libraries:
   - org.apache.parquet: parquet-avro
 
 Review comment:
   Apologies for the churn, but it looks like maybe the updated version adds a new dependency which wasn't previously in the list:
   
   ```
   Error: found 1 missing licenses. These licenses are reported, but missing in the registry
   druid_module: parquet-extensions, groupId: org.apache.parquet, artifactId: parquet-format-structures, version: 1.11.0, license: Apache License version 2.0
   ```
   
   I think if you add `parquet-format-structures` to this list of `libraries` then it should be ok?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org