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

Beam JIRA Bot commented on BEAM-2069:
-------------------------------------

This issue is P2 but has been unassigned without any comment for 60 days so it 
has been labeled "stale-P2". If this issue is still affecting you, we care! 
Please comment and remove the label. Otherwise, in 14 days the issue will be 
moved to P3.

Please see https://beam.apache.org/contribute/jira-priorities/ for a detailed 
explanation of what these priorities mean.


> Remove ResourceId.getCurrentDirectory()?
> ----------------------------------------
>
>                 Key: BEAM-2069
>                 URL: https://issues.apache.org/jira/browse/BEAM-2069
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>    Affects Versions: 2.0.0
>            Reporter: Stephen Sisk
>            Priority: P2
>              Labels: backward-incompatible, stale-P2
>
> Beam ResourceId 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 ResourceId 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
(v8.3.4#803005)

Reply via email to