[jira] [Comment Edited] (FLINK-26061) FileSystem.getFileStatus fails for directories

["Matthias Pohl (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FLINK-26061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17490733#comment-17490733 ] 

Matthias Pohl edited comment on FLINK-26061 at 2/11/22, 8:14 AM:
-----------------------------------------------------------------

This issue was already addressed in FLINK-8373. But it's not addressed properly:

Creating an empty directory and checking for existence does not work in the presto implementation. See corresponding [FileSystemBehaviorTestSuite:125|https://github.com/apache/flink/blob/c6997c97c575d334679915c328792b8a3067cfb5/flink-core/src/test/java/org/apache/flink/core/fs/FileSystemBehaviorTestSuite.java#L125] where this specific behavior is excluded.

We might want to fix that properly since it's an easy-to-fall-into trap


was (Author: mapohl):
This issue was already addressed in FLINK-8373. But it's not addressed properly:

Creating an empty directory and checking for existence does not work in the presto implementation.

> FileSystem.getFileStatus fails for directories
> ----------------------------------------------
>
>                 Key: FLINK-26061
>                 URL: https://issues.apache.org/jira/browse/FLINK-26061
>             Project: Flink
>          Issue Type: Technical Debt
>          Components: Connectors / FileSystem
>    Affects Versions: 1.15.0
>            Reporter: Matthias Pohl
>            Priority: Major
>
> {{FileSystem.getFileStatus}} is not supported for directories in object stores (like s3). We might want to adjust the interface here. It's not clear based on the contract that for certain {{FileSystem}} implementations. This is especially unintuitive because there is a {{isDir()}} method provided by the resulting {{FileStatus}} instance.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)