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

Andrew Wang reassigned HADOOP-9818:
-----------------------------------

    Assignee: Kousuke Saruta

Hi Kousuke, thanks for the patch!

I've made you a contributor, so now you should be able to assign Common JIRAs 
to yourself and mark them as Patch Available via the "Submit Patch" button.

Part of the trickiness from removing "2>&1" is error handling. I haven't looked 
into DF at all, but {{ProcessBuilder}} separates stdout and stderr, so stderr 
messages no longer show up. So, I added another param to {{Shell}} in 
HADOOP-9652 (hopefully committed soon) which merges them back together.

It'd be great if you can review the {{DF}} error handling behavior, especially 
adding tests for any missing holes you find.
                
> Remove usage of "bash -c" from oah.fs.DF
> ----------------------------------------
>
>                 Key: HADOOP-9818
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9818
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 3.0.0, 2.1.0-beta
>            Reporter: Andrew Wang
>            Assignee: Kousuke Saruta
>              Labels: newbie
>         Attachments: HADOOP-9818.patch
>
>
> {{DF}} uses "bash -c" to shell out to the unix {{df}} command. This is 
> potentially unsafe; let's think about removing it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to