[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Akira Ajisaka (JIRA)

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

Akira Ajisaka commented on HADOOP-14156:


LGTM, +1.

> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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



[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HADOOP-14156:


| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
14s{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 
18s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m 
27s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
39s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green}  1m 
13s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
24s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  1m 
30s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
54s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m  
9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 10m  
9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green}  1m  
4s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
24s{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 
38s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
53s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  8m 
20s{color} | {color:green} hadoop-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
42s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 54m 38s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker |  Image:yetus/hadoop:a9ad5d6 |
| JIRA Issue | HADOOP-14156 |
| GITHUB PR | https://github.com/apache/hadoop/pull/187 |
| Optional Tests |  asflicense  compile  javac  javadoc  mvninstall  mvnsite  
unit  findbugs  checkstyle  |
| uname | Linux ef7368c26c6e 3.13.0-103-generic #150-Ubuntu SMP Thu Nov 24 
10:34:17 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh 
|
| git revision | trunk / 385d2cb |
| Default Java | 1.8.0_121 |
| findbugs | v3.0.0 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HADOOP-Build/11792/testReport/ |
| modules | C: hadoop-common-project/hadoop-common U: 
hadoop-common-project/hadoop-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HADOOP-Build/11792/console |
| Powered by | Apache Yetus 0.5.0-SNAPSHOT   http://yetus.apache.org |


This message was automatically generated.



> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should 

[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Akira Ajisaka (JIRA)

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

Akira Ajisaka commented on HADOOP-14156:


bq. I'll file a jira to Apache Yetus to fix this problem.
Filed YETUS-494.

> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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



[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Akira Ajisaka (JIRA)

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

Akira Ajisaka commented on HADOOP-14156:


Submitted precommit job: 
https://builds.apache.org/job/PreCommit-HADOOP-Build/11791/
Probably it works.

> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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



[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Akira Ajisaka (JIRA)

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

Akira Ajisaka commented on HADOOP-14156:


There is a workaround: comment the URL to the patch: 
https://github.com/apache/hadoop/pull/187.patch

> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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



[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-09 Thread Akira Ajisaka (JIRA)

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

Akira Ajisaka commented on HADOOP-14156:


Umm... smart-apply-patch is not working. 
https://builds.apache.org/job/PreCommit-HADOOP-Build/11790/console
Normally PR is created after creating JIRA, and then ASF GitHub bot comments 
the URL to the patch here. However, this issue was created after the 
corresponding PR, so the bot did not comment the URL to the patch. Therefore 
smart-apply-patch does not work for this issue because smart-apply-patch search 
if there is a patch URL. I'll file a jira to Apache Yetus to fix this problem.

> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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



[jira] [Commented] (HADOOP-14156) Grammar error in the ConfTest.java

2017-03-08 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on HADOOP-14156:
-

Github user adyatlov commented on the issue:

https://github.com/apache/hadoop/pull/187
  
Thank you, @aajisaka. Done.


> Grammar error in the ConfTest.java
> --
>
> Key: HADOOP-14156
> URL: https://issues.apache.org/jira/browse/HADOOP-14156
> Project: Hadoop Common
>  Issue Type: Bug
>  Components: test
>Reporter: Andrey Dyatlov
>Priority: Trivial
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> In the file 
> {{hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ConfTest.java}}
> bq. does not defined
> should be replaced by
> bq. is not defined
> PR: https://github.com/apache/hadoop/pull/187/



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

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