[jira] [Commented] (HDFS-15428) Javadocs fails for hadoop-federation-balance

2020-06-22 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17141938#comment-17141938
 ] 

Hudson commented on HDFS-15428:
---

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #18373 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/18373/])
HDFS-15428. Javadocs fails for hadoop-federation-balance. Contributed by 
(aajisaka: rev 201d734af3992df13bc5f4d47b8869da4fb2b2c5)
* (edit) 
hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java
* (edit) 
hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedureScheduler.java


> Javadocs fails for hadoop-federation-balance
> 
>
> Key: HDFS-15428
> URL: https://issues.apache.org/jira/browse/HDFS-15428
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>  Components: documentation
>Reporter: Xieming Li
>Assignee: Xieming Li
>Priority: Minor
> Fix For: 3.4.0
>
> Attachments: HADOOP-17080.000.patch
>
>
> javadoc fails for hadoop-federation-balance
> {code:java}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-javadoc-plugin:3.0.1:jar (module-javadocs) on 
> project hadoop-federation-balance: MavenReportException: Error while 
> generating Javadoc:
> [ERROR] Exit code: 1 - 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public abstract boolean execute() throws RetryException, 
> IOException;
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:129:
>  warning: no @return
> [ERROR]   public long delayMillisBeforeRetry() {
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:136:
>  warning: no @return
> [ERROR]   protected boolean isSchedulerShutdown() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:151:
>  warning: no @return
> [ERROR]   public String nextProcedure() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:158:
>  warning: no @return
> [ERROR]   public String name() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/DistCpProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public DistCpProcedure(String name, String nextProcedure, long 
> delayDuration,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:46:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:src path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:48:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:dst path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for name
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for nextProcedure
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for delayDuration
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for dstNs
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> 

[jira] [Commented] (HDFS-15428) Javadocs fails for hadoop-federation-balance

2020-06-22 Thread Hadoop QA (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17141849#comment-17141849
 ] 

Hadoop QA commented on HDFS-15428:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  1m 
18s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} dupname {color} | {color:green}  0m  
0s{color} | {color:green} No case conflicting files found. {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:brown} trunk Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 21m 
27s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
22s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
17s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green}  0m 
23s{color} | {color:green} trunk passed {color} |
| {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 
16m 34s{color} | {color:green} branch has no errors when building and testing 
our client artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
21s{color} | {color:green} trunk passed {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue}  0m 
35s{color} | {color:blue} Used deprecated FindBugs config; considering 
switching to SpotBugs. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
32s{color} | {color:green} trunk passed {color} |
|| || || || {color:brown} Patch Compile Tests {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 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvnsite {color} | {color:green}  0m 
19s{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} shadedclient {color} | {color:green} 
15m 30s{color} | {color:green} patch has no errors when building and testing 
our client artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
17s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
39s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m  
5s{color} | {color:green} hadoop-federation-balance in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
27s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 62m  1s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | ClientAPI=1.40 ServerAPI=1.40 base: 
https://builds.apache.org/job/PreCommit-HDFS-Build/29449/artifact/out/Dockerfile
 |
| JIRA Issue | HDFS-15428 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/13006163/HADOOP-17080.000.patch
 |
| Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite 
unit shadedclient findbugs checkstyle |
| uname | Linux 8d9da567b966 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 
10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | personality/hadoop.sh |
| git revision | trunk / ce1008fe61a |
| Default Java | Private Build-1.8.0_252-8u252-b09-1~18.04-b09 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HDFS-Build/29449/testReport/ |
| Max. process+thread count | 309 (vs. ulimit of 5500) |
| 

[jira] [Commented] (HDFS-15428) Javadocs fails for hadoop-federation-balance

2020-06-22 Thread Xieming Li (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17141812#comment-17141812
 ] 

Xieming Li commented on HDFS-15428:
---

[~aajisaka]
Thank you for the review.

> Javadocs fails for hadoop-federation-balance
> 
>
> Key: HDFS-15428
> URL: https://issues.apache.org/jira/browse/HDFS-15428
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>Reporter: Xieming Li
>Assignee: Xieming Li
>Priority: Minor
> Attachments: HADOOP-17080.000.patch
>
>
> javadoc fails for hadoop-federation-balance
> {code:java}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-javadoc-plugin:3.0.1:jar (module-javadocs) on 
> project hadoop-federation-balance: MavenReportException: Error while 
> generating Javadoc:
> [ERROR] Exit code: 1 - 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public abstract boolean execute() throws RetryException, 
> IOException;
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:129:
>  warning: no @return
> [ERROR]   public long delayMillisBeforeRetry() {
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:136:
>  warning: no @return
> [ERROR]   protected boolean isSchedulerShutdown() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:151:
>  warning: no @return
> [ERROR]   public String nextProcedure() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:158:
>  warning: no @return
> [ERROR]   public String name() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/DistCpProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public DistCpProcedure(String name, String nextProcedure, long 
> delayDuration,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:46:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:src path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:48:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:dst path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for name
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for nextProcedure
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for delayDuration
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for dstNs
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @throws for java.io.IOException
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/TrashProcedure.java:52:
>  warning: no @throws for java.io.IOException
> [ERROR]   public TrashProcedure(String name, String nextProcedure, long 

[jira] [Commented] (HDFS-15428) Javadocs fails for hadoop-federation-balance

2020-06-22 Thread Akira Ajisaka (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17141804#comment-17141804
 ] 

Akira Ajisaka commented on HDFS-15428:
--

Moved.

+1 for the patch.

> Javadocs fails for hadoop-federation-balance
> 
>
> Key: HDFS-15428
> URL: https://issues.apache.org/jira/browse/HDFS-15428
> Project: Hadoop HDFS
>  Issue Type: Sub-task
>Reporter: Xieming Li
>Assignee: Xieming Li
>Priority: Minor
> Attachments: HADOOP-17080.000.patch
>
>
> javadoc fails for hadoop-federation-balance
> {code:java}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-javadoc-plugin:3.0.1:jar (module-javadocs) on 
> project hadoop-federation-balance: MavenReportException: Error while 
> generating Javadoc:
> [ERROR] Exit code: 1 - 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public abstract boolean execute() throws RetryException, 
> IOException;
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:129:
>  warning: no @return
> [ERROR]   public long delayMillisBeforeRetry() {
> [ERROR]   ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:136:
>  warning: no @return
> [ERROR]   protected boolean isSchedulerShutdown() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:151:
>  warning: no @return
> [ERROR]   public String nextProcedure() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/procedure/BalanceProcedure.java:158:
>  warning: no @return
> [ERROR]   public String name() {
> [ERROR] ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/DistCpProcedure.java:124:
>  warning: no @throws for java.io.IOException
> [ERROR]   public DistCpProcedure(String name, String nextProcedure, long 
> delayDuration,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:46:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:src path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:48:
>  error: bad use of '>'
> [ERROR]  *   /a/b/c -> {ns:dst path:/a/b/c}
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for name
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for nextProcedure
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for delayDuration
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @param for dstNs
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/MountTableProcedure.java:66:
>  warning: no @throws for java.io.IOException
> [ERROR]   public MountTableProcedure(String name, String nextProcedure,
> [ERROR]  ^
> [ERROR] 
> /Users/sri/projects/hadoop-mirror/hadoop-tools/hadoop-federation-balance/src/main/java/org/apache/hadoop/tools/fedbalance/TrashProcedure.java:52:
>  warning: no @throws for java.io.IOException
> [ERROR]   public TrashProcedure(String name, String nextProcedure, long 
>