[jira] [Commented] (TEZ-3205) Update tez poms to remove hadoop 2.2/2.4 profiles

2016-07-11 Thread Siddharth Seth (JIRA) 

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

Siddharth Seth commented on TEZ-3205:
-

Good points. I'll update the patch when I get a chance.

> Update tez poms to remove hadoop 2.2/2.4 profiles
> -
>
> Key: TEZ-3205
> URL: https://issues.apache.org/jira/browse/TEZ-3205
> Project: Apache Tez
>  Issue Type: Bug
>Reporter: Siddharth Seth
> Attachments: TEZ-3205.01.patch
>
>




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

[jira] [Commented] (TEZ-3205) Update tez poms to remove hadoop 2.2/2.4 profiles

2016-07-09 Thread Hitesh Shah (JIRA) 

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

Hitesh Shah commented on TEZ-3205:
--

Comments:
  - Given that hadoop 2.6.0 is the default now, there should be no need for a 
hadoop26 profile in various poms ( such as 
tez-plugins/tez-yarn-timeline-history/ and tez-tools/analyzers/ ).
  - Doesnt this mean that the default shim when nothing is specified should be 
the 2.6 impl i.e. get rid of the 2.6 shim to and replace with the default impl 
with 2.6 specific code?
  -  Seems like there is no need for profiles in tez-tools/analyzers/pom.xml as 
job-analyzer should always be compiled from now on? 

Might be better to do these code changes only on master given that initial 
releases of 0.8 did allow builds against 2.2/2.4 even if not fully supported.

> Update tez poms to remove hadoop 2.2/2.4 profiles
> -
>
> Key: TEZ-3205
> URL: https://issues.apache.org/jira/browse/TEZ-3205
> Project: Apache Tez
>  Issue Type: Bug
>Reporter: Siddharth Seth
> Attachments: TEZ-3205.01.patch
>
>




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

[jira] [Commented] (TEZ-3205) Update tez poms to remove hadoop 2.2/2.4 profiles

2016-07-09 Thread TezQA (JIRA) 

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

TezQA commented on TEZ-3205:


{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment
  http://issues.apache.org/jira/secure/attachment/12817007/TEZ-3205.01.patch
  against master revision 608e15e.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 3.0.1) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-TEZ-Build/1844//testReport/
Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/1844//console

This message is automatically generated.

> Update tez poms to remove hadoop 2.2/2.4 profiles
> -
>
> Key: TEZ-3205
> URL: https://issues.apache.org/jira/browse/TEZ-3205
> Project: Apache Tez
>  Issue Type: Bug
>Reporter: Siddharth Seth
> Attachments: TEZ-3205.01.patch
>
>




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