This is an automated email from the ASF dual-hosted git repository.

shoothzj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 3e224a2ac4 Fix javadoc and build warnings (#3297)
3e224a2ac4 is described below

commit 3e224a2ac40b8cd7804c758a195e70a9b55cfdaa
Author: Vladislav Khakin <[email protected]>
AuthorDate: Tue May 31 02:47:45 2022 +0300

    Fix javadoc and build warnings (#3297)
    
    Fix #472 #2155
---
 .../main/java/org/apache/bookkeeper/client/BKException.java  |  2 +-
 .../java/org/apache/bookkeeper/conf/ClientConfiguration.java |  4 ++--
 .../java/org/apache/bookkeeper/conf/ServerConfiguration.java | 12 ++++++------
 tests/scripts/pom.xml                                        |  1 +
 4 files changed, 10 insertions(+), 9 deletions(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java
index 7435c67edf..8f917d7944 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java
@@ -479,7 +479,7 @@ public abstract class BKException extends 
org.apache.bookkeeper.client.api.BKExc
     /**
      * Extract an exception code from an BKException, or use a default if it's 
another type.
      * The throwable is null, assume that no exception took place and return
-     * {@link BKException.Code.OK}.
+     * {@link BKException.Code#OK}.
      */
     public static int getExceptionCode(Throwable t, int defaultCode) {
         if (t == null) {
diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java
index 935445174f..f77714fec4 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java
@@ -1153,10 +1153,10 @@ public class ClientConfiguration extends 
AbstractConfiguration<ClientConfigurati
      * Enable/disable reordering read sequence on reading entries.
      *
      * <p>If this flag is enabled, the client will use
-     * {@link EnsemblePlacementPolicy#reorderReadSequence(java.util.ArrayList,
+     * {@link EnsemblePlacementPolicy#reorderReadSequence(java.util.List,
      * org.apache.bookkeeper.client.BookiesHealthInfo, 
org.apache.bookkeeper.client.DistributionSchedule.WriteSet)}
      * to figure out a better read sequence to attempt reads from replicas and 
use
-     * {@link 
EnsemblePlacementPolicy#reorderReadLACSequence(java.util.ArrayList,
+     * {@link EnsemblePlacementPolicy#reorderReadLACSequence(java.util.List,
      * org.apache.bookkeeper.client.BookiesHealthInfo, 
org.apache.bookkeeper.client.DistributionSchedule.WriteSet)}
      * to figure out a better read sequence to attempt long poll reads from 
replicas.
      *
diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
index 9ce055610d..3101dbbe37 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
@@ -562,14 +562,14 @@ public class ServerConfiguration extends 
AbstractConfiguration<ServerConfigurati
     /**
      * Get local scrub interval.
      *
-     * @return Number of seconds between scrubs, <= 0 for disabled.
+     * @return Number of seconds between scrubs, {@literal <=}0 for disabled.
      */
     public long getLocalScrubPeriod() {
         return this.getLong(LOCAL_SCRUB_PERIOD, 0);
     }
 
     /**
-     * Set local scrub period in seconds (<= 0 for disabled). Scrub will be 
scheduled at delays
+     * Set local scrub period in seconds ({@literal <=}0 for disabled). Scrub 
will be scheduled at delays
      * chosen from the interval (.5 * interval, 1.5 * interval)
      */
     public void setLocalScrubPeriod(long period) {
@@ -1680,7 +1680,7 @@ public class ServerConfiguration extends 
AbstractConfiguration<ServerConfigurati
     }
 
     /**
-     * Get the maximum milliseconds to run major compaction. If <= 0 the
+     * Get the maximum milliseconds to run major compaction. If {@literal <=}0 
the
      * thread will run until all compaction is completed.
      *
      * @return limit
@@ -1691,7 +1691,7 @@ public class ServerConfiguration extends 
AbstractConfiguration<ServerConfigurati
     }
 
     /**
-     * Set the maximum milliseconds to run major compaction. If <= 0 the
+     * Set the maximum milliseconds to run major compaction. If {@literal <=}0 
the
      * thread will run until all compaction is completed.
      *
      * @see #getMajorCompactionMaxTimeMillis()
@@ -1757,7 +1757,7 @@ public class ServerConfiguration extends 
AbstractConfiguration<ServerConfigurati
     }
 
     /**
-     * Get the maximum milliseconds to run minor compaction. If <= 0 the
+     * Get the maximum milliseconds to run minor compaction. If {@literal <=}0 
the
      * thread will run until all compaction is completed.
      *
      * @return limit
@@ -1768,7 +1768,7 @@ public class ServerConfiguration extends 
AbstractConfiguration<ServerConfigurati
     }
 
     /**
-     * Set the maximum milliseconds to run minor compaction. If <= 0 the
+     * Set the maximum milliseconds to run minor compaction. If {@literal <=}0 
the
      * thread will run until all compaction is completed.
      *
      * @see #getMinorCompactionMaxTimeMillis()
diff --git a/tests/scripts/pom.xml b/tests/scripts/pom.xml
index 907d47d0a9..6a3e614d50 100644
--- a/tests/scripts/pom.xml
+++ b/tests/scripts/pom.xml
@@ -55,6 +55,7 @@
       <plugin>
         <groupId>org.codehaus.mojo</groupId>
         <artifactId>exec-maven-plugin</artifactId>
+        <version>${exec-maven-plugin.version}</version>
         <executions>
           <execution>
             <id>bash-tests</id>

Reply via email to