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 2026/02/23 07:40:16 UTC

[PR] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

jbonofre opened a new pull request, #3861:
URL: https://github.com/apache/polaris/pull/3861

   This closes #2484 


-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on code in PR #3861:
URL: https://github.com/apache/polaris/pull/3861#discussion_r2839744168


##########
runtime/admin/distribution/LICENSE:
##########
@@ -202,11 +202,24 @@
 
 --------------------------------------------------------------------------------
 
-This binary artifact bundles the following projects:
+This propduct bundles and includes code from Apache Iceberg.

Review Comment:
   bundle -> dependencies
   includes -> code copied into Polaris



-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #3861:
URL: https://github.com/apache/polaris/pull/3861#discussion_r2839815842


##########
runtime/admin/distribution/LICENSE:
##########
@@ -202,11 +202,24 @@
 
 --------------------------------------------------------------------------------
 
-This binary artifact bundles the following projects:
+This propduct bundles and includes code from Apache Iceberg.

Review Comment:
   Correct 😀 



-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on PR #3861:
URL: https://github.com/apache/polaris/pull/3861#issuecomment-3992455823

   Closing as the change has been merged as part of #3881 


-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3861:
URL: https://github.com/apache/polaris/pull/3861#discussion_r2839725304


##########
runtime/admin/distribution/LICENSE:
##########
@@ -202,11 +202,24 @@
 
 --------------------------------------------------------------------------------
 
-This binary artifact bundles the following projects:
+This propduct bundles and includes code from Apache Iceberg.

Review Comment:
   What is the difference between "bundles and includes" vs just "bundles"?



-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on code in PR #3861:
URL: https://github.com/apache/polaris/pull/3861#discussion_r2839744168


##########
runtime/admin/distribution/LICENSE:
##########
@@ -202,11 +202,24 @@
 
 --------------------------------------------------------------------------------
 
-This binary artifact bundles the following projects:
+This propduct bundles and includes code from Apache Iceberg.

Review Comment:
   bundle -> dependencies from a project
   includes -> code copied into Polaris



-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on PR #3861:
URL: https://github.com/apache/polaris/pull/3861#issuecomment-3944756432

   This change is included in #3864


-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3861:
URL: https://github.com/apache/polaris/pull/3861#discussion_r2841242801


##########
runtime/admin/distribution/LICENSE:
##########
@@ -446,30 +406,22 @@ License: BSD 3-Clause
 
 --------------------------------------------------------------------------------
 
-Group: com.google.api.grpc Name: gapic-google-cloud-storage-v2 Version: 2.60.0
-Group: com.google.api.grpc Name: grpc-google-cloud-storage-v2 Version: 2.60.0
-Group: com.google.api.grpc Name: proto-google-cloud-monitoring-v3 Version: 3.52.0
-Group: com.google.api.grpc Name: proto-google-cloud-storage-v2 Version: 2.60.0
+This product bundles Storage APIs.

Review Comment:
   ```suggestion
   This product bundles Google gRPC Storage APIs.
   ```



-- 
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] chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy closed pull request #3861: chore: remove versions from LICENSE and NOTICE files to "relax" dependecy updates, improving also the wording
URL: https://github.com/apache/polaris/pull/3861


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