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

Jie Yu commented on MESOS-3951:
-------------------------------

Cleanup patches:

commit 1aaf1c3c98e6da68372f7fb761a356fb4219b20d
Author: Jie Yu <[email protected]>
Date:   Thu Nov 19 12:21:13 2015 -0800

    Used factory method to create HDFS client.
    
    Review: https://reviews.apache.org/r/40498

commit d6781a4e9a4f361a98e50980d3ed0a6ab4350208
Author: Jie Yu <[email protected]>
Date:   Wed Nov 18 16:03:30 2015 -0800

    Fixed a few style issues in HDFS wrapper code.
    
    Review: https://reviews.apache.org/r/40464

commit 5749b80c1452bcdb8e972d162dfe2d1096109c5e
Author: Jie Yu <[email protected]>
Date:   Wed Nov 18 15:55:50 2015 -0800

    Moved HDFS wrapper implementation to a cpp file.
    
    Review: https://reviews.apache.org/r/40463

commit 9c231b2b1551b15d2ca805c193bba7df2f8dfc39
Author: Jie Yu <[email protected]>
Date:   Wed Nov 18 15:30:11 2015 -0800

    Fixed the license header in hdfs.hpp.
    
    Review: https://reviews.apache.org/r/40462

commit fefb79f4bb9b94da66a27f30958ed96440510b5a
Author: Jie Yu <[email protected]>
Date:   Wed Nov 18 15:28:57 2015 -0800

    Changed HDFS wrapper from a struct to a class.
    
    Review: https://reviews.apache.org/r/40461

> Make HDFS tool wrappers asynchronous.
> -------------------------------------
>
>                 Key: MESOS-3951
>                 URL: https://issues.apache.org/jira/browse/MESOS-3951
>             Project: Mesos
>          Issue Type: Task
>            Reporter: Jie Yu
>
> The existing HDFS tool wrappers (src/hdfs/hdfs.hpp) are synchronous. They use 
> os::shell to shell out the 'hadoop' commands. This makes it very hard to be 
> reused at other locations in the code base.
> The URI fetcher HDFS plugin will try to re-use the existing HDFS tool 
> wrappers. In order to do that, we need to make it asynchronous first.



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

Reply via email to