Update Spark Plugin BUNDLE-LICENSE/NOTICE for 1.4.0#3909
Update Spark Plugin BUNDLE-LICENSE/NOTICE for 1.4.0#3909snazy wants to merge 3 commits intoapache:mainfrom
Conversation
jbonofre
left a comment
jbonofre
left a comment
There was a problem hiding this comment.
There are a few mistakes in the PR.
In the meantime, I'm doing a full pass on the spark plugin (uber jar).
|
|
||
| This binary artifact contains Apache Hive. | ||
|
|
||
| Copyright: 2008-2020 The Apache Software Foundation |
There was a problem hiding this comment.
Just a note: copyright and home page are not strictly required in LICENSE.
|
|
||
| -------------------------------------------------------------------------------- | ||
|
|
||
| This binary artifact contains fast_float. |
There was a problem hiding this comment.
This one is a duplicate from previous one.
| This binary artifact contains fast_float. | ||
|
|
||
| Project URL: https://github.com/fastfloat/fast_float | ||
| License: MIT - https://github.com/fastfloat/fast_float/blob/35d523195bf7d57aba0e735ad6eba1e6f71ba8d6/LICENSE-MIT |
There was a problem hiding this comment.
Instead of the link, for BSD and MIT licenses, we have to copy the license content inline here.
|
|
||
| -------------------------------------------------------------------------------- | ||
|
|
||
| This binary artifact contains code from Kite, developed at Cloudera, Inc. with |
There was a problem hiding this comment.
It's covered by the Iceberg NOTICE.
| This binary artifact includes Google Protobuf with the following copyright | ||
| notice: | ||
|
|
||
| | Copyright 2008 Google Inc. All rights reserved. |
There was a problem hiding this comment.
This is a BSD license. So the license content should be inline in the LICENSE, not in NOTICE.
The NOTICE should only contains NOTICE from Apache license, and only Apache license, dependencies.
| https://github.com/locationtech/jts/blob/master/LICENSES.md | ||
| https://github.com/geotools/geotools/wiki/JTS-Shapefile-Contribution | ||
|
|
||
| | Copyright (c) 2003, Open Source Geospatial Foundation |
There was a problem hiding this comment.
It's a BSD license, so the license content should be in the LICENSE, not the NOTICE.
|
|
||
| This binary artifact contains code from FastDoubleParser with the following license: | ||
|
|
||
| | # FastDoubleParser |
There was a problem hiding this comment.
This is a MIT license, so license content should be in LICENSE file, not in NOTICE.
|
|
||
| This binary artifact contains code from fast_float with the following license: | ||
|
|
||
| | Third-party license for |
There was a problem hiding this comment.
This is a MIT license, so the license content should be in the LICENSE file, not the NOTICE file.
|
|
||
| This binary artifact contains code from bigint with the following license: | ||
|
|
||
| | Third-party license for |
There was a problem hiding this comment.
This is a BSD license, so the license content should be in the LICENSE file, not in the NOTICE file.
|
@jbonofre I've pushed an update via the The 3rd commit is a different approach by "just" copying the contents of the |
2 participants
Fixes #3873