lordk911 commented on a change in pull request #1058:
URL: https://github.com/apache/incubator-linkis/pull/1058#discussion_r740696777



##########
File path: 
linkis-commons/linkis-common/src/main/java/org/apache/linkis/common/io/Fs.java
##########
@@ -52,4 +52,8 @@
 
     public abstract boolean renameTo(FsPath oldDest, FsPath newDest) throws 
IOException;
 
+    public abstract boolean mkdir(FsPath dest) throws IOException;

Review comment:
       interface Fs in module linkis-common have a direct implement abstract 
class FileSystem in module linkis-storage. The method add into Fs have already 
IN FileSystem  before this PR

##########
File path: 
linkis-commons/linkis-common/src/main/java/org/apache/linkis/common/io/Fs.java
##########
@@ -52,4 +52,8 @@
 
     public abstract boolean renameTo(FsPath oldDest, FsPath newDest) throws 
IOException;
 
+    public abstract boolean mkdir(FsPath dest) throws IOException;

Review comment:
       I have remove the the abstract method mkdir\mkdirs from abstract class 
FileSystem and lift them to interface




-- 
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: dev-unsubscr...@linkis.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@linkis.apache.org
For additional commands, e-mail: dev-h...@linkis.apache.org

Reply via email to