dimas-b commented on code in PR #2430:
URL: https://github.com/apache/polaris/pull/2430#discussion_r2293985780
##########
spec/iceberg-rest-catalog-open-api.yaml:
##########
@@ -79,7 +79,7 @@ paths:
required: false
schema:
type: string
- description: Warehouse location or identifier to request from the
service
+ description: Warehouse location (Catalog name) or identifier to
request from the service
Review Comment:
In general, we should not edit this file. It is owned and maintained under
the Iceberg project. Cf.
https://github.com/apache/polaris/blob/1e8dc887d691fa1c7b45ed3ae17a4475b384056a/spec/README.md
##########
spec/polaris-management-service.yml:
##########
@@ -28,10 +28,10 @@ servers:
variables:
scheme:
description: The scheme of the URI, either http or https.
- default: https
+ default: http
Review Comment:
Why default to `http` (without TLS)?
--
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]
