You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "wsjz (via GitHub)" <gi...@apache.org> on 2023/04/28 12:29:54 UTC

[GitHub] [doris] wsjz commented on a diff in pull request #19098: [refactor](fs)(step3)use filesystem instead of old storage, new storage just access remote object storage

wsjz commented on code in PR #19098:
URL: https://github.com/apache/doris/pull/19098#discussion_r1180345905


##########
fe/fe-core/src/main/java/org/apache/doris/fs/FileSystemFactory.java:
##########
@@ -33,7 +38,7 @@ public static FileSystem get(StorageBackend.StorageType type, Map<String, String
         return get(type.name(), type, properties);
     }
 
-    public static FileSystem get(String name, StorageBackend.StorageType type, Map<String, String> properties) {
+    public static RemoteFileSystem get(String name, StorageBackend.StorageType type, Map<String, String> properties) {

Review Comment:
   Can not remove it for now, it denpendent on Stmt



-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org