[automatic-failover] Fix formatting for anticipated AA Failover changes (#4183)

* fix formatting for anticipated failover changes

* additional formatting to cover given folders

Author: atakavci

hbase-formatter.xml

@@ -40,7 +40,7 @@
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_while" value="insert"/>
 <setting id="org.eclipse.jdt.core.formatter.blank_lines_after_package" value="1"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_type_parameters" value="insert"/>
-<setting id="org.eclipse.jdt.core.formatter.continuation_indentation" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.continuation_indentation" value="1"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_after_postfix_operator" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_method_invocation" value="20"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_arguments" value="do not insert"/>
@@ -74,15 +74,15 @@
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_type_parameters" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_block_comment" value="true"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_type_declaration" value="insert"/>
-<setting id="org.eclipse.jdt.core.formatter.lineSplit" value="100"/>
+<setting id="org.eclipse.jdt.core.formatter.lineSplit" value="120"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_if" value="insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_between_brackets_in_array_type_reference" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_parenthesized_expression" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_explicitconstructorcall_arguments" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_constructor_declaration" value="insert"/>
 <setting id="org.eclipse.jdt.core.formatter.blank_lines_before_first_class_body_declaration" value="0"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_method" value="insert"/>
-<setting id="org.eclipse.jdt.core.formatter.indentation.size" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.indentation.size" value="4"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_declaration" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.enabling_tag" value="@formatter:on"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_enum_constant" value="do not insert"/>
@@ -145,7 +145,7 @@
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_declaration_parameters" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_superinterfaces" value="insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_allocation_expression" value="do not insert"/>
-<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="4"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_type_reference" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_opening_brace_in_array_initializer" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_brace_in_block" value="insert"/>
@@ -205,7 +205,7 @@
 <setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_while_in_do_statement" value="do not insert"/>
 <setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_enum_constant" value="16"/>
 <setting id="org.eclipse.jdt.core.formatter.comment.format_javadoc_comments" value="true"/>
-<setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="100"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="120"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_package" value="insert"/>
 <setting id="org.eclipse.jdt.core.formatter.blank_lines_between_import_groups" value="1"/>
 <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_enum_constant_arguments" value="do not insert"/>

pom.xml

@@ -333,11 +333,18 @@
 				<configuration>
 					<configFile>${project.basedir}/hbase-formatter.xml</configFile>
 					<directories>
-						<directory>${project.basedir}/src/main/java/redis/clients/jedis/annots</directory>
-						<directory>${project.basedir}/src/main/java/redis/clients/jedis/mcf</directory>
-						<directory>${project.basedir}/src/test/java/redis/clients/jedis/misc/AutomaticFailoverTest.java</directory>
-						<directory>${project.basedir}/src/test/java/redis/clients/jedis/failover</directory>
+						<directory>${project.basedir}</directory>
 					</directories>
+					<includes>
+						<!-- Specific files -->
+						<include>src/main/java/redis/clients/jedis/annots/*.java</include>
+						<include>src/main/java/redis/clients/jedis/mcf/*.java</include>
+						<include>src/test/java/redis/clients/jedis/failover/*.java</include>
+						<include>src/test/java/redis/clients/jedis/misc/AutomaticFailoverTest.java</include>
+						<include>src/main/java/redis/clients/jedis/providers/MultiClusterPooledConnectionProvider.java</include>
+						<include>src/main/java/redis/clients/jedis/MultiClusterClientConfig.java</include>
+						<include>src/main/java/redis/clients/jedis/HostAndPort.java</include>
+					</includes>
 				</configuration>
 				<executions>
 					<execution>

src/main/java/redis/clients/jedis/HostAndPort.java

@@ -4,52 +4,52 @@
 
 public class HostAndPort implements Serializable {
 
-  private static final long serialVersionUID = -519876229978427751L;
-
-  private final String host;
-  private final int port;
-
-  public HostAndPort(String host, int port) {
-    this.host = host;
-    this.port = port;
-  }
-
-  public String getHost() {
-    return host;
-  }
-
-  public int getPort() {
-    return port;
-  }
-
-  @Override
-  public boolean equals(Object obj) {
-    if (obj == null) return false;
-    if (obj == this) return true;
-    if (!(obj instanceof HostAndPort)) return false;
-    HostAndPort other = (HostAndPort) obj;
-    return this.port == other.port && this.host.equals(other.host);
-  }
-
-  @Override
-  public int hashCode() {
-    return 31 * host.hashCode() + port;
-  }
-
-  @Override
-  public String toString() {
-    return host + ":" + port;
-  }
-
-  /**
-   * Creates HostAndPort with <i>unconverted</i> host.
-   * @param string String to parse. Must be in <b>"host:port"</b> format. Port is mandatory.
-   * @return parsed HostAndPort
-   */
-  public static HostAndPort from(String string) {
-    int lastColon = string.lastIndexOf(":");
-    String host = string.substring(0, lastColon);
-    int port = Integer.parseInt(string.substring(lastColon + 1));
-    return new HostAndPort(host, port);
-  }
+    private static final long serialVersionUID = -519876229978427751L;
+
+    private final String host;
+    private final int port;
+
+    public HostAndPort(String host, int port) {
+        this.host = host;
+        this.port = port;
+    }
+
+    public String getHost() {
+        return host;
+    }
+
+    public int getPort() {
+        return port;
+    }
+
+    @Override
+    public boolean equals(Object obj) {
+        if (obj == null) return false;
+        if (obj == this) return true;
+        if (!(obj instanceof HostAndPort)) return false;
+        HostAndPort other = (HostAndPort) obj;
+        return this.port == other.port && this.host.equals(other.host);
+    }
+
+    @Override
+    public int hashCode() {
+        return 31 * host.hashCode() + port;
+    }
+
+    @Override
+    public String toString() {
+        return host + ":" + port;
+    }
+
+    /**
+     * Creates HostAndPort with <i>unconverted</i> host.
+     * @param string String to parse. Must be in <b>"host:port"</b> format. Port is mandatory.
+     * @return parsed HostAndPort
+     */
+    public static HostAndPort from(String string) {
+        int lastColon = string.lastIndexOf(":");
+        String host = string.substring(0, lastColon);
+        int port = Integer.parseInt(string.substring(lastColon + 1));
+        return new HostAndPort(host, port);
+    }
 }