You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Stephen Sisk (JIRA)" <ji...@apache.org> on 2017/04/24 23:26:04 UTC

[jira] [Updated] (BEAM-2069) Remove FileSystem.getCurrentDirectory()?

     [ https://issues.apache.org/jira/browse/BEAM-2069?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stephen Sisk updated BEAM-2069:
-------------------------------
    Affects Version/s: First stable release

> Remove FileSystem.getCurrentDirectory()?
> ----------------------------------------
>
>                 Key: BEAM-2069
>                 URL: https://issues.apache.org/jira/browse/BEAM-2069
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>    Affects Versions: First stable release
>            Reporter: Stephen Sisk
>            Assignee: Davor Bonaci
>              Labels: backward-incompatible
>
> Beam FileSystem currently has a getCurrentDirectory method that returns the current resource id if it's a directory, or the parent directory if it's a directory.
> To implement this you need to know whether or not a particular path is a directory or not.
> I'm trying to implement the Hadoop FileSystem implementation, and it's not clear if it's possible. Hadoop's Paths do not end a / if they are a directory (they are stripped), nor do hadoop paths tell you if something is a directory, so it's not possible to determine if a given path is a file that does not have a suffix, or a directory.
> It's not clear to me that all file systems can determine whether a path is a directory and thus I don't believe it can be implemented reliably.
> The only usages of getCurrentDirectory that I could find are in tests so it's not clear we actually need this.
> I propose that we remove this method.
> cc [~davor]



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)