andrew4699 commented on code in PR #1159: URL: https://github.com/apache/polaris/pull/1159#discussion_r1989988974
########## service/common/src/main/java/org/apache/polaris/service/exception/IcebergExceptionMapper.java: ########## @@ -68,6 +68,8 @@ public class IcebergExceptionMapper implements ExceptionMapper<RuntimeException> /** Signifies that we could not extract an HTTP code from a given cloud exception */ public static final int UNKNOWN_CLOUD_HTTP_CODE = -1; + @VisibleForTesting public static final String AZURE_STORAGE_URL_SUFFIX = ".blob.core.windows.net"; Review Comment: What are the other ones? We could add them upfront if folks know about them or do it iteratively. -- 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]
