Fix checkstyle

Author: lhotari

bookkeeper-server/src/main/java/org/apache/bookkeeper/zookeeper/CustomZooKeeperHostProvider.java

@@ -39,7 +39,7 @@
 import org.slf4j.LoggerFactory;
 
 /**
- * ZooKeeper HostProvider with a workaround for https://issues.apache.org/jira/browse/ZOOKEEPER-3825
+ * ZooKeeper HostProvider with a workaround for https://issues.apache.org/jira/browse/ZOOKEEPER-3825.
  * Based on ZooKeeper's StaticHostProvider
  */
 public final class CustomZooKeeperHostProvider implements HostProvider {
@@ -53,7 +53,7 @@ public final class CustomZooKeeperHostProvider implements HostProvider {
     private int currentIndex = -1;
 
     /**
-     * The following fields are used to migrate clients during reconfiguration
+     * The following fields are used to migrate clients during reconfiguration.
      */
     private boolean reconfigMode = false;
 
@@ -87,7 +87,8 @@ public CustomZooKeeperHostProvider(Collection<InetSocketAddress> serverAddresses
     /**
      * Constructs a SimpleHostSet.
      *
-     * Introduced for testing purposes. getAllByName() is a static method of InetAddress, therefore cannot be easily mocked.
+     * Introduced for testing purposes. getAllByName() is a static method of InetAddress, therefore cannot be easily
+     * mocked.
      * By abstraction of Resolver interface we can easily inject a mocked implementation in tests.
      *
      * @param serverAddresses
@@ -100,8 +101,8 @@ public CustomZooKeeperHostProvider(Collection<InetSocketAddress> serverAddresses
     }
 
     /**
-     * Constructs a SimpleHostSet. This constructor is used from StaticHostProviderTest to produce deterministic test results
-     * by initializing sourceOfRandomness with the same seed
+     * Constructs a SimpleHostSet. This constructor is used from StaticHostProviderTest to produce deterministic test
+     * results by initializing sourceOfRandomness with the same seed
      *
      * @param serverAddresses
      *            possibly unresolved ZooKeeper server addresses
@@ -127,7 +128,8 @@ private void init(Collection<InetSocketAddress> serverAddresses, long randomness
     private InetSocketAddress resolve(InetSocketAddress address) {
         try {
             String curHostString = address.getHostString();
-            List<InetAddress> resolvedAddresses = new ArrayList<>(Arrays.asList(this.resolver.getAllByName(curHostString)));
+            List<InetAddress> resolvedAddresses =
+                    new ArrayList<>(Arrays.asList(this.resolver.getAllByName(curHostString)));
             if (resolvedAddresses.isEmpty()) {
                 return address;
             }

tests/integration-tests-utils/src/main/java/org/apache/bookkeeper/tests/integration/utils/CustomZooKeeperHostProvider.java

@@ -38,7 +38,7 @@
 import org.slf4j.LoggerFactory;
 
 /**
- * ZooKeeper HostProvider with a workaround for https://issues.apache.org/jira/browse/ZOOKEEPER-3825
+ * ZooKeeper HostProvider with a workaround for https://issues.apache.org/jira/browse/ZOOKEEPER-3825.
  * Based on ZooKeeper's StaticHostProvider
  */
 public final class CustomZooKeeperHostProvider implements HostProvider {
@@ -52,7 +52,7 @@ public final class CustomZooKeeperHostProvider implements HostProvider {
     private int currentIndex = -1;
 
     /**
-     * The following fields are used to migrate clients during reconfiguration
+     * The following fields are used to migrate clients during reconfiguration.
      */
     private boolean reconfigMode = false;
 
@@ -86,7 +86,8 @@ public CustomZooKeeperHostProvider(Collection<InetSocketAddress> serverAddresses
     /**
      * Constructs a SimpleHostSet.
      *
-     * Introduced for testing purposes. getAllByName() is a static method of InetAddress, therefore cannot be easily mocked.
+     * Introduced for testing purposes. getAllByName() is a static method of InetAddress, therefore cannot be easily
+     * mocked.
      * By abstraction of Resolver interface we can easily inject a mocked implementation in tests.
      *
      * @param serverAddresses
@@ -99,8 +100,8 @@ public CustomZooKeeperHostProvider(Collection<InetSocketAddress> serverAddresses
     }
 
     /**
-     * Constructs a SimpleHostSet. This constructor is used from StaticHostProviderTest to produce deterministic test results
-     * by initializing sourceOfRandomness with the same seed
+     * Constructs a SimpleHostSet. This constructor is used from StaticHostProviderTest to produce deterministic test
+     * results by initializing sourceOfRandomness with the same seed
      *
      * @param serverAddresses
      *            possibly unresolved ZooKeeper server addresses
@@ -126,7 +127,8 @@ private void init(Collection<InetSocketAddress> serverAddresses, long randomness
     private InetSocketAddress resolve(InetSocketAddress address) {
         try {
             String curHostString = address.getHostString();
-            List<InetAddress> resolvedAddresses = new ArrayList<>(Arrays.asList(this.resolver.getAllByName(curHostString)));
+            List<InetAddress> resolvedAddresses =
+                    new ArrayList<>(Arrays.asList(this.resolver.getAllByName(curHostString)));
             if (resolvedAddresses.isEmpty()) {
                 return address;
             }