You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "jbonofre (via GitHub)" <gi...@apache.org> on 2025/01/28 10:29:59 UTC
[PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
jbonofre opened a new pull request, #903:
URL: https://github.com/apache/polaris/pull/903
This PR adds the missing parts in LICENSE/NOTICE found in 0.9.0 rc4.
@rdblue @snazy
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1933538163
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
@snazy thanks for the update, it helps.
@rdblue imho, we are in the same situate as for Gradle resources (e.g. `gradlew`, ...): as the code is copied from another project, this project is under the ALv2, so we should document where the code is coming and the copyright.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1933550795
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
Side note: `gradle/wrapper/gradle-wrapper.properties` is a _generated_ file, not "copied" from anywhere.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932662996
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
Review Comment:
Yup, it's would be cleaner 😄 Doing the change.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #903:
URL: https://github.com/apache/polaris/pull/903#issuecomment-2628816652
I updated Nessie `NOTICE`. I think we are good for Polaris.
@rdblue do you mind to do a new pass ? Thanks !
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765782
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + EPL.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE, but I don't think it's a blocker (I guess @snazy already fixed for next nessie release).
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765651
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + CE.
It would be great if Nessie clarify in the NOTICE, but I don't think it's a blocker.
@rdblue thoughts ?
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + CE.
It would be great if Nessie clarify in the NOTICE, but I don't think it's a blocker.
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936764783
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
Review Comment:
Fair. Let's include this for now (I will discuss with the community after the release).
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932667326
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
Review Comment:
I think that it's good to have the doc in the source distribution (it's very valuable for users). I would propose to keep it for now, and remove from `LICENSE` *if* we remove it later from the source distribution.
@rdblue wdyt ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1933538504
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
See https://github.com/apache/polaris/pull/903#discussion_r1933538163
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932660427
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
But, I don't know if we are talking about the same original authors. @snazy can you confirm that you are the author of the original files ? or do we have other dremio contributors on these files ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932658919
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
Were these not contributed to the project by Dremio employees? If that's the case, then I think the regular contribution rules would apply, even if the code is present in another project. It doesn't matter that it was originally in another project, it matters that it was directly contributed by the copyright owner. If the copyright owner was not the one to contribute, then the third-party documentation would be needed.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1943587972
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I've updated Nessie `NOTICE` with the 0.102.5 version.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932663290
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
Correct, I asked the question on the other comment.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936472909
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I guess it's fine to treat it as a third-party contribution if the attribution is unknown from the Nessie project.
I'm actually more concerned right now about the GPL reference that popped up in the Iceberg NOTICE changes.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1935633026
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765782
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + EPL.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE.
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936703612
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
@rdblue yup. That's a concern to me too. Let me check.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1943588263
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
I've updated Nessie NOTICE with the 0.102.5 version.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre merged PR #903:
URL: https://github.com/apache/polaris/pull/903
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932659891
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
This addition depends on whether the code was contributed or copied as third-party content. My understanding is that it was contributed directly by Dremio and this should not be present.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #903:
URL: https://github.com/apache/polaris/pull/903#issuecomment-2639009108
I think we are good now. I propose to merge this PR and I will submit rc5 to vote.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1933517675
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
There are multiple authors on this code
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765651
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + CE.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE, but I don't think it's a blocker (I guess @snazy already fixed for next nessie release).
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936469487
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
@jbonofre I think we need to verify that there is no GPL code in here, given the mention of GPL code in the NOTICE updates to Iceberg.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932656835
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
Review Comment:
If this is going to be removed in the next release, why document it rather than removing it before 0.9.0?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1932660261
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
Review Comment:
Newline for the second copyright?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936468475
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
Review Comment:
I don't think it is a blocker as long as the code is documented.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765651
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + EPL.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE, but I don't think it's a blocker (I guess @snazy already fixed for next nessie release).
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765651
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
+
+* site/layouts/community/list.html
+* site/layouts/docs/baseof.html
+* site/layouts/partials/community_links.html
+* site/layouts/partials/head.html
+* site/layouts/partials/navbar.html
+* site/layouts/shortcodes/redoc-polaris.html
+
+Home page: https://www.docsy.dev/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Project Nessie.
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + EPL.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE.
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1936765782
##########
NOTICE:
##########
@@ -6,3 +6,9 @@ The Apache Software Foundation (http://www.apache.org/).
The initial code for the Polaris project was donated
to the ASF by Snowflake Inc. (https://www.snowflake.com/) copyright 2024.
+
+Apache Iceberg
+Copyright 2017-2025 The Apache Software Foundation
+
+Dremio
+Copyright 2015-2017 Dremio Corporation
Review Comment:
I checked and it's dual license: CDDL + GPL or GPL + EPL.
I checked in https://github.com/projectnessie/nessie/releases/download/nessie-0.102.2/nessie-aggregated-license-report-0.102.2.zip
It would be great if Nessie clarify in the NOTICE.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add the missing parts (code coming from other projects) in LICENSE & NOTICE [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #903:
URL: https://github.com/apache/polaris/pull/903#discussion_r1935633304
##########
LICENSE:
##########
@@ -211,3 +211,96 @@ Home page: https://github.com/gradle/gradle
License: https://www.apache.org/licenses/LICENSE-2.0
--------------------------------------------------------------------------------
+
+This product includes code from Apache Iceberg.
+
+* spec/rest-catalog-open-api.yaml
+* integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationTest.java
+
+Copyright: Copyright 2017-2025 The Apache Software Foundation
+Home page: https://iceberg.apache.org
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from OpenAPITool openapi-generator
+
+* server-templates/formParams.mustache
+* server-templates/apiService.mustache
+* server-templates/bodyParams.mustache
+* server-templates/pojo.mustache
+* server-templates/headerParams.mustache
+* server-templates/apiServiceImpl.mustache
+* server-templates/queryParams.mustache
+* server-templates/api.mustache
+
+Copyright: Copyright 2018 OpenAPI-Generator Contributors (https://openapi-generator.tech) Copyright 2018 SmartBear Software
+Home page: https://openapi-generator.tech/
+License: https://www.apache.org/licenses/LICENSE-2.0
+
+--------------------------------------------------------------------------------
+
+This product includes code from Google Docsy.
Review Comment:
@rdblue thoughts ?
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org