zentol commented on a change in pull request #18692:
URL: https://github.com/apache/flink/pull/18692#discussion_r803512647
##########
File path: flink-core/src/main/java/org/apache/flink/core/fs/FileSystem.java
##########
@@ -684,7 +686,9 @@ public boolean exists(final Path f) throws IOException {
/**
* Make the given file and all non-existent parents into directories. Has
the semantics of Unix
- * 'mkdir -p'. Existence of the directory hierarchy is not an error.
+ * 'mkdir -p'. Existence of the directory hierarchy is not an error.
{@code FileSystem}
+ * implementations being backed by an object store like S3 do not support
directories. No
+ * entities are going to be created in that case.
Review comment:
This is only true for presto; the hadoop s3 filesystem creates marker
files.
--
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...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
