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

Sreenath Somarajapuram commented on TEZ-3470:
---------------------------------------------

[~hitesh]
bq. Can you clarify a bit more on this? Are you saying the tez-ui UTs get 
executed even when mvn test is not called? As an example, the tez-ui pom 
clearly indicates that the frontend plugin is being invoked in the 
generate-resources phase.
- Thats true, without skip UTs would get executed on running "mvn clean package"
- https://github.com/eirslett/frontend-maven-plugin/issues/113 is what I could 
find regarding the behavior in frontend-maven-plugin.
- And as mentioned @ 
https://github.com/eirslett/frontend-maven-plugin#running-karma, the phase work 
only on using karma (goal=npm). Karma is a different test runner.
- phase=test is not working with goal=npm, which we need to customize npm 
install.

bq. There is no mention on why we need a flag on needing to skip tests in this 
jira. Can you explain why the changes are needed? I see TEZ-3482 as a related 
jira but does not seem to have mentioned in the previous comment.
- skip just accept one property name. The flag was created to have one property 
that could be used.

bq. why do we need so many profiles: skipUTsOnSkipTests and 
skipUTsOnMavenTestSkip ? This seems like a bad design.
- Totally agree, but because of the above this is the best way I could find.

bq. Assuming that the skip tests are needed because tests are not working in 
some envs, having that be done in a separate jira allows the patch to be 
removed once all issues are fixed. Doing everything in this one jira makes the 
revert complex.
- Have created TEZ-3492 for adding a profile to disable UI UTs.

> Tez UI: Make the build work in IBM PPC
> --------------------------------------
>
>                 Key: TEZ-3470
>                 URL: https://issues.apache.org/jira/browse/TEZ-3470
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.8.3
>            Reporter: Sreenath Somarajapuram
>            Assignee: Sreenath Somarajapuram
>              Labels: TezUI
>         Attachments: TEZ-3470.1.patch, TEZ-3470.2.patch, TEZ-3470.3.patch, 
> TEZ-3470.wip.1.patch
>
>
> Current versions of frontend-maven-plugin, node & npm used by the build is 
> not functioning as expected in IBM PPC. Following version works, and the 
> build must be changed to used the same in a PPC.
> - frontend-maven-plugin : v1.1
> - node : v5.7.0
> - npm : 3.6.0



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to