spaces-X commented on code in PR #65379:
URL: https://github.com/apache/doris/pull/65379#discussion_r3543603116
##########
fe/fe-core/src/main/java/org/apache/doris/common/util/S3Util.java:
##########
@@ -434,6 +430,13 @@ public static void validateAndTestEndpoint(String
endpoint) throws UserException
}
}
+ static String buildEndpointUrl(String endpoint) {
Review Comment:
One compatibility question: for users on HTTP-only storage with a scheme-less
endpoint like `minio.local:9000`, would existing loads start failing after
upgrade
since it now defaults to HTTPS. If so, might be worth a
release-note callout
--
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]
