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

Daryn Sharp updated HADOOP-7224:
--------------------------------

    Attachment: HADOOP-7224-2.patch

(I'm just updating comments in the patch)

The factory has a registerCommands(Class) method in order to abstract the 
factory's contract with its commands.  Doing so will allow the api to evolve 
w/o impacting the caller (such as FsShell).

I eventually plan to add a method to FsShell to return the registrar class so 
subclasses can return their own class for commands.  These subclasses will be 
insulated from the details of how and when the factory is initialized.

> Add command factory to FsShell
> ------------------------------
>
>                 Key: HADOOP-7224
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7224
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 0.22.0
>            Reporter: Daryn Sharp
>            Assignee: Daryn Sharp
>         Attachments: HADOOP-7224-2.patch, HADOOP-7224.patch
>
>
> The FsShell has many chains if/then/else chains for instantiating and running 
> commands.  A dynamic mechanism is needed for registering commands such that 
> FsShell requires no changes when adding new commands.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to