You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@reef.apache.org by "Julia (JIRA)" <ji...@apache.org> on 2015/10/26 18:27:27 UTC

[jira] [Commented] (REEF-822) Improve UriPrefix API in IFileSystem

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

Julia commented on REEF-822:
----------------------------

A recent integration testing shows when we get a file path, it already contains uriPrefix. We should really hide UriPrefix property from the IFileSystem, instead, add a method Uri CreateUriForPath(string path). I am going to make the change. 

> Improve UriPrefix API in IFileSystem
> ------------------------------------
>
>                 Key: REEF-822
>                 URL: https://issues.apache.org/jira/browse/REEF-822
>             Project: REEF
>          Issue Type: Bug
>            Reporter: Julia
>            Assignee: Julia
>
> A proposal in the code review for REEF-538 it to change IFileSystem API
> string UriPrefix { get; } 
> into a method like
> Uri CreateUriForPath(string path)
> This is to hid UriPrefix  from the caller. We should also check if prefix is already in the path given so that  to make the code robust.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)