[ 
https://issues.apache.org/jira/browse/HADOOP-9246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13577782#comment-13577782
 ] 

Chris Nauroth commented on HADOOP-9246:
---------------------------------------

Hi Alejandro,

The addendum patch would re-break HADOOP-9245 (mvn clean without running mvn 
install before fails).  Without specifying phase explicitly in the pom.xml, 
Maven needs to look at the mojo definition to determine default phase.  
Unfortunately, if you don't have hadoop-maven-plugins installed yet, then there 
is no mojo definition to inspect, and mvn clean fails.

Of course, we can keep the change to add defaultPhase in the mojo code.  I 
think we just also need to keep <phase> in the pom.xml files.  +1 for just the 
change in {{VersionInfoMojo}}.

Thanks!

                
> Execution phase for hadoop-maven-plugin should be process-resources
> -------------------------------------------------------------------
>
>                 Key: HADOOP-9246
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9246
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: build
>    Affects Versions: 3.0.0, trunk-win
>            Reporter: Karthik Kambatla
>            Assignee: Karthik Kambatla
>             Fix For: 3.0.0
>
>         Attachments: HADOOP-9246.2.patch, HADOOP-9246-addendum.patch, 
> hadoop-9246.patch, hadoop-9246.patch
>
>
> Per discussion on HADOOP-9245, the execution phase of hadoop-maven-plugin 
> should be _process-resources_ and not _compile_.

--
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