[
https://issues.apache.org/jira/browse/HADOOP-7377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13049351#comment-13049351
]
Matt Foley commented on HADOOP-7377:
------------------------------------
I'm uncomfortable using reflection with access permission override, to get a
datum that seems like it could be fetched using provided methods such as
getCommandName(). However, Daryn has convinced me that it would require a
massive change of legacy code to make this better:
bq. [Daryn] The CommandFactory is supposed to call setName(...) on the instance
it creates, using the name fed to the factory (since commands can have more
than one invokable name). setName()/getName() should be just using the
instance field name. The static NAME is legacy and was not intended to remain.
But Count and the DFSAdmin commands, unfortunately, use it. I intend to kill
both getCommandName() and NAME, if possible, after DFSAdmin is adapted to
better use the framework. Basically, the command should not know it's name,
that's the factory's job, and the factory tells the command how it was conjured
up. But in the meantime, NAME is needed.
So, +1 on this patch.
Auto-test passes except for new test, but this patch will be tested by existing
DFSAdmin unit tests starting to work.
I'll commit this in 24 hours to allow for other comments.
Daryn, please list the exact unit tests that will be fixed by this submission.
Thanks.
> Fix command name handling affecting DFSAdmin
> --------------------------------------------
>
> Key: HADOOP-7377
> URL: https://issues.apache.org/jira/browse/HADOOP-7377
> Project: Hadoop Common
> Issue Type: Bug
> Components: fs
> Affects Versions: 0.23.0
> Reporter: Daryn Sharp
> Assignee: Daryn Sharp
> Attachments: HADOOP-7377.patch
>
>
> When an error occurs in the get/set quota commands in DFSAdmin, they are
> displaying the following:
> setQuota: failed to get SetQuotaCommand.NAME
> The {{Command}} class expects the {{NAME}} field to be accessible, but for
> DFSAdmin, it's not.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
