CalvinKirs commented on code in PR #56861:
URL: https://github.com/apache/doris/pull/56861#discussion_r2434466207
##########
fe/fe-core/src/main/java/org/apache/doris/datasource/property/storage/AzureProperties.java:
##########
@@ -155,24 +159,46 @@ public static String formatAzureEndpoint(String endpoint,
String accessKey) {
@Override
public String validateAndNormalizeUri(String url) throws UserException {
- return S3PropertyUtils.validateAndNormalizeUri(url, usePathStyle,
forceParsingByStandardUrl);
+ return AzurePropertyUtils.validateAndNormalizeUri(url);
}
@Override
public String validateAndGetUri(Map<String, String> loadProps) throws
UserException {
- return S3PropertyUtils.validateAndGetUri(loadProps);
+ return AzurePropertyUtils.validateAndGetUri(loadProps);
}
@Override
public String getStorageName() {
- return "Azure";
+ return "AZURE";
Review Comment:
we’ve updated the logic to keep Storage names fully uppercase for
consistency, since both HDFS and S3 follow that convention. All callers already
perform case-insensitive matching, so this change ensures uniform style without
affecting compatibility.
--
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]
