[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-29 Thread Akira Ajisaka (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16028238#comment-16028238
 ] 

Akira Ajisaka commented on MAPREDUCE-6887:
--

+1, the test failures are not related.

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha4
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch, MAPREDUCE-6887.002.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-29 Thread Akira Ajisaka (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16028196#comment-16028196
 ] 

Akira Ajisaka commented on MAPREDUCE-6887:
--

>From 
>https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6984/artifact/patchprocess/jiracommentfile

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 20s 
{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s 
{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 
0s {color} | {color:green} The patch appears to include 13 new or modified test 
files. {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 8s 
{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 13m 
14s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 1m 58s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
34s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 42s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 
51s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 3m 
49s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 45s 
{color} | {color:green} trunk passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 9s 
{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 2m 
21s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 4s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 2m 4s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
30s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 28s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 
41s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 
0s {color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 4m 
30s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 30s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 49s 
{color} | {color:green} hadoop-mapreduce-client-core in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 43s 
{color} | {color:green} hadoop-mapreduce-client-common in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 9m 23s 
{color} | {color:green} hadoop-mapreduce-client-app in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 3m 24s 
{color} | {color:green} hadoop-mapreduce-client-hs in the patch passed. {color} 
|
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 118m 53s 
{color} | {color:red} hadoop-mapreduce-client-jobclient in the patch failed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 37s 
{color} | {color:green} hadoop-mapreduce-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 
26s {color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 179m 13s {color} 
| {color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | hadoop.mapreduce.TestMRJobClient |
| Timed out junit tests | 
org.apache.hadoop.mapreduce.lib.jobcontrol.TestMapReduceJobControl |
\\
\\
|| Subsystem || Report/Notes ||
| Docker |  Image:yetus/hadoop:14b5c93 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12869820/MAPREDUCE-6887.002.patch
 |
| JIRA Issue | MAPREDUCE-6887 |
| Optional Tests |

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024595#comment-16024595
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Through the log,Unit test failure is not caused by this patch

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch, MAPREDUCE-6887.002.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread Hadoop QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024572#comment-16024572
 ] 

Hadoop QA commented on MAPREDUCE-6887:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 20s 
{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s 
{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 
0s {color} | {color:green} The patch appears to include 13 new or modified test 
files. {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 8s 
{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 13m 
14s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 1m 58s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
34s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 42s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 
51s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 3m 
49s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 45s 
{color} | {color:green} trunk passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 9s 
{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 2m 
21s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 2m 4s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 2m 4s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
30s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 28s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 
41s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 
0s {color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 4m 
30s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 1m 30s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 49s 
{color} | {color:green} hadoop-mapreduce-client-core in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 43s 
{color} | {color:green} hadoop-mapreduce-client-common in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 9m 23s 
{color} | {color:green} hadoop-mapreduce-client-app in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 3m 24s 
{color} | {color:green} hadoop-mapreduce-client-hs in the patch passed. {color} 
|
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 118m 53s 
{color} | {color:red} hadoop-mapreduce-client-jobclient in the patch failed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 37s 
{color} | {color:green} hadoop-mapreduce-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 
26s {color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 179m 13s {color} 
| {color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | hadoop.mapreduce.TestMRJobClient |
| Timed out junit tests | 
org.apache.hadoop.mapreduce.lib.jobcontrol.TestMapReduceJobControl |
\\
\\
|| Subsystem || Report/Notes ||
| Docker |  Image:yetus/hadoop:14b5c93 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12869820/MAPREDUCE-6887.002.patch
 |
| JIRA Issue | MAPREDUCE-6887 |
| Optional Tests |  asflicense  compile  javac  javadoc  mvninstall  mvnsite  
unit  findbugs  checkstyle  |
| uname | Linux 63920f524a19

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024470#comment-16024470
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Hi,[~ajisakaa],I go through the whole project and find some similar 
problems,Can you plz review them?

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch, MAPREDUCE-6887.002.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024386#comment-16024386
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

[~ajisakaa],I submit a new patch

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch, MAPREDUCE-6887.002.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024371#comment-16024371
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

OK

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-25 Thread Akira Ajisaka (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16024366#comment-16024366
 ] 

Akira Ajisaka commented on MAPREDUCE-6887:
--

The change looks good. Hi [~linzhangbing], would you remove redundant 'static' 
in other places as well in hadoop-mapreduce?

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Assignee: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-14 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009927#comment-16009927
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Hi,[~rohithsharma],
I can't change the Assignee to oneself also when submit the YARN-6577.Please 
add me as contributor for Apache Hadoop.My account is ZhangBing Lin.
Thanks
ZhangBing Lin

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the 'static' modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-11 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16007571#comment-16007571
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Just delete the redundant codes, no unit tests needed!

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the static modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-11 Thread Hadoop QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16007562#comment-16007562
 ] 

Hadoop QA commented on MAPREDUCE-6887:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 30s 
{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s 
{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s 
{color} | {color:red} 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} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 12m 
41s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 28s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
16s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 32s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 
15s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 
52s {color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s 
{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 
24s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 23s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 23s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
16s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 28s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 
13s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 
0s {color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 2s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 59s 
{color} | {color:green} hadoop-mapreduce-client-core in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 
20s {color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 23m 2s {color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker |  Image:yetus/hadoop:14b5c93 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12867703/MAPREDUCE-6887.001.patch
 |
| JIRA Issue | MAPREDUCE-6887 |
| Optional Tests |  asflicense  compile  javac  javadoc  mvninstall  mvnsite  
unit  findbugs  checkstyle  |
| uname | Linux 5e00c42d0ffe 4.4.0-43-generic #63-Ubuntu SMP Wed Oct 12 
13:48:03 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh 
|
| git revision | trunk / 0d5c8ed |
| Default Java | 1.8.0_121 |
| findbugs | v3.1.0-RC1 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6976/testReport/ |
| modules | C: 
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core 
U: 
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core |
| Console output | 
https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6976/console |
| Powered by | Apache Yetus 0.3.0   http://yetus.apache.org |


This message was automatically generated.



> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Priority:

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-11 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16007554#comment-16007554
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Hi,[~ajisakaa],
I can't change the Assignee to oneself also when submit the 
MAPREDUCE-6887.Please add me as contributor for Apache Hadoop Map/Reduce.My 
account is ZhangBing Lin.
   Thanks
   ZhangBing Lin

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the static modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

[jira] [Commented] (MAPREDUCE-6887) Modifier 'static' is redundant for inner enums less

2017-05-11 Thread ZhangBing Lin (JIRA) 

[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16007542#comment-16007542
 ] 

ZhangBing Lin commented on MAPREDUCE-6887:
--

Submit a patch!

> Modifier 'static' is redundant for inner enums less
> ---
>
> Key: MAPREDUCE-6887
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6887
> Project: Hadoop Map/Reduce
>  Issue Type: Bug
>Affects Versions: 3.0.0-alpha3
>Reporter: ZhangBing Lin
>Priority: Minor
> Attachments: MAPREDUCE-6887.001.patch
>
>
>   Java enumeration type is a static constant, implicitly modified with static 
> final,Modifier 'static' is redundant for inner enums less.So I suggest 
> deleting the static modifier.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org