[
https://issues.apache.org/jira/browse/HADOOP-12930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15272755#comment-15272755
]
Allen Wittenauer commented on HADOOP-12930:
-------------------------------------------
Just to copy what I sent to common-dev here:
===
When the sub-projects re-merged, maven work was done, whatever, the
shell scripts for MR and YARN were placed (effectively) outside of the normal
maven hierarchy. In order to add unit tests to the shell scripts for these
sub-projects, it means effectively turning hadoop-yarn-project/hadoop-yarn and
hadoop-mapreduce-project into “real” modules so that mvn test works as
expected. Doing so will likely have some surprising consequences, such as
anyone who modifies java code and the shell code in a patch will trigger _all_
of the unit tests in yarn.
I think we have four options:
a) Continue forward turning these into real modules with src directories, etc
and we live with the consequences
b) Move the related bits into an existing module, making them similar to HDFS,
common, tools
c) Move the related bits into a new module, using the layout that maven really
really wants
d) Skip the unit tests; we don’t have them now
This is clearly more work than what I really wanted to cover in this
branch, but given that there was a specific request to add unit test code for
this functionality, I’m sort of stuck here.
Thoughts?
===
> [Umbrella] Dynamic subcommands for hadoop shell scripts
> -------------------------------------------------------
>
> Key: HADOOP-12930
> URL: https://issues.apache.org/jira/browse/HADOOP-12930
> Project: Hadoop Common
> Issue Type: Bug
> Components: scripts
> Affects Versions: 3.0.0
> Reporter: Allen Wittenauer
> Assignee: Allen Wittenauer
> Priority: Critical
>
> Umbrella for converting hadoop, hdfs, mapred, and yarn to allow for dynamic
> subcommands. See first comment for more details.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
