jbonofre commented on code in PR #16043: URL: https://github.com/apache/iceberg/pull/16043#discussion_r3135822880
########## spark/v4.0/spark-runtime/LICENSE: ########## @@ -207,71 +207,71 @@ This binary artifact contains Apache Avro. Copyright: Copyright 2010-2019 The Apache Software Foundation Home page: https://avro.apache.org/ -License: http://www.apache.org/licenses/LICENSE-2.0 +License: Apache License, Version 2.0 - http://www.apache.org/licenses/LICENSE-2.0.txt -------------------------------------------------------------------------------- This binary artifact contains the Jackson JSON processor. Copyright: 2007-2019 Tatu Saloranta and other contributors Home page: http://jackson.codehaus.org/ -License: http://www.apache.org/licenses/LICENSE-2.0.txt +License: Apache License, Version 2.0 - http://www.apache.org/licenses/LICENSE-2.0.txt -------------------------------------------------------------------------------- This binary artifact contains Apache Parquet. Copyright: 2014-2024 The Apache Software Foundation Home page: https://parquet.apache.org/ -License: http://www.apache.org/licenses/LICENSE-2.0 +License: Apache License, Version 2.0 - http://www.apache.org/licenses/LICENSE-2.0.txt Review Comment: That makes sense to me (and it was my initial change 😄 ). I propose to take a step back here with the following: 1. Let me list the dependencies in each "modules" (bundle and runtime artifacts) with a first pass for dependencies requiring "discussions" (I'm thinking of Eclipse Collections for instance). 2. I'm updating this PR with the minimal changes to have LICENSE/NOTICE files correct. 3. Depending of our finding on 1, I will update the PR accordingly. Does it work for you guys? Thanks again for your help on this one! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
