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

Boris Shkolnik updated HADOOP-4708:
-----------------------------------

    Status: Patch Available  (was: Open)

recommended fixes to the patch:
1. moving logic of selecting correct function for a type of command  to 
CommandExecutor (I think it makes sense to do it now, cause it is more clean)
2. making get methods final
3. removing unnecessary formating and casting.
4. changed to switch from if-else with a default throw.

I left getCmd and not toString() - it makes more sense. 

> Add support for dfsadmin commands for test TestCLI unit test
> ------------------------------------------------------------
>
>                 Key: HADOOP-4708
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4708
>             Project: Hadoop Core
>          Issue Type: New Feature
>          Components: test
>            Reporter: Boris Shkolnik
>            Assignee: Boris Shkolnik
>             Fix For: 0.20.0
>
>         Attachments:  HADOOP-4708.patch, HADOOP-4708-1.patch
>
>
> Curretly TestCLI assumes dfs command when describing tests in testConf.xml
> we need to add ability to run dfsadmin commands too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to