dimas-b commented on code in PR #1604:
URL: https://github.com/apache/polaris/pull/1604#discussion_r2102837479
##########
polaris-core/src/main/java/org/apache/polaris/core/storage/StorageLocation.java:
##########
@@ -44,8 +44,10 @@ protected StorageLocation(@Nonnull String location) {
this.location = URI.create(location.replaceFirst("file:/+",
LOCAL_PATH_PREFIX)).toString();
} else if (location.startsWith("/")) {
this.location = URI.create(location.replaceFirst("/+",
LOCAL_PATH_PREFIX)).toString();
- } else {
+ } else if (location.startsWith(LOCAL_PATH_PREFIX)) {
Review Comment:
This line of code is gone now, but I think the question about upper case
URIs remains. Since the purpose of this PR is directly related to that, I'm ok
with addressing upper case URIs later.
--
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-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
