stevenzwu commented on code in PR #16215:
URL: https://github.com/apache/iceberg/pull/16215#discussion_r3196512568


##########
spark/v4.1/spark-runtime/LICENSE:
##########
@@ -345,6 +436,385 @@ Copyright: 2006-2019 The Guava Authors
 Project URL: https://github.com/google/guava
 License: Apache License, Version 2.0 - 
https://www.apache.org/licenses/LICENSE-2.0
 
+This product bundles the Mozilla Public Suffix List, distributed by Apache 
HttpComponents.

Review Comment:
   Re-flagging — the thread is marked resolved but the missing separator is 
still present. In `spark/v3.5/spark-runtime/LICENSE`, 
`spark/v4.0/spark-runtime/LICENSE`, and `spark/v4.1/spark-runtime/LICENSE` the 
new Mozilla PSL entry was inserted directly after Guava without the standard 
`--------------------------------------------------------------------------------`
 separator (around line 435, between the Guava `License:` line and the `This 
product bundles the Mozilla Public Suffix List` header). v3.4 is fine because 
PSL was placed elsewhere in that file.



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

Reply via email to