fix java code style

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/auth/AnonymousAuthenticationProviderImpl.java

@@ -26,7 +26,7 @@
 public class AnonymousAuthenticationProviderImpl implements PasswdAuthenticationProvider {
 
   @Override
-  public void Authenticate(String user, String password) throws AuthenticationException {
+  public void authenticate(String user, String password) throws AuthenticationException {
     // no-op authentication
   }
 

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/auth/CustomAuthenticationProviderImpl.java

@@ -43,8 +43,8 @@ public class CustomAuthenticationProviderImpl implements PasswdAuthenticationPro
   }
 
   @Override
-  public void Authenticate(String user, String password) throws AuthenticationException {
-    customProvider.Authenticate(user, password);
+  public void authenticate(String user, String password) throws AuthenticationException {
+    customProvider.authenticate(user, password);
   }
 
 }

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/auth/LdapAuthenticationProviderImpl.java

@@ -40,7 +40,7 @@ public class LdapAuthenticationProviderImpl implements PasswdAuthenticationProvi
   }
 
   @Override
-  public void Authenticate(String user, String password) throws AuthenticationException {
+  public void authenticate(String user, String password) throws AuthenticationException {
 
     Hashtable<String, Object> env = new Hashtable<String, Object>();
     env.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.ldap.LdapCtxFactory");

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/auth/PamAuthenticationProviderImpl.java

@@ -32,7 +32,7 @@ public class PamAuthenticationProviderImpl implements PasswdAuthenticationProvid
   }
 
   @Override
-  public void Authenticate(String user, String password) throws AuthenticationException {
+  public void authenticate(String user, String password) throws AuthenticationException {
 
     if (pamServiceNames == null || pamServiceNames.trim().isEmpty()) {
       throw new AuthenticationException("No PAM services are set.");

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/auth/PasswdAuthenticationProvider.java

@@ -22,7 +22,7 @@
 public interface PasswdAuthenticationProvider {
 
   /**
-   * The Authenticate method is called by the HiveServer2 authentication layer
+   * The authenticate method is called by the HiveServer2 authentication layer
    * to authenticate users for their requests.
    * If a user is to be granted, return nothing/throw nothing.
    * When a user is to be disallowed, throw an appropriate {@link AuthenticationException}.
@@ -35,5 +35,5 @@ public interface PasswdAuthenticationProvider {
    * @throws AuthenticationException When a user is found to be
    *                                 invalid by the implementation
    */
-  void Authenticate(String user, String password) throws AuthenticationException;
+  void authenticate(String user, String password) throws AuthenticationException;
 }

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/cli/GetInfoType.java

@@ -86,7 +86,8 @@ public static GetInfoType getGetInfoType(TGetInfoType tGetInfoType) {
                 return infoType;
             }
         }
-        throw new IllegalArgumentException("Unrecognized Thrift TGetInfoType value: " + tGetInfoType);
+        throw new IllegalArgumentException("Unrecognized Thrift TGetInfoType value: "
+                + tGetInfoType);
     }
 
     public TGetInfoType toTGetInfoType() {

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/cli/SparkThriftServerSQLException.java

@@ -97,7 +97,10 @@ public SparkThriftServerSQLException(String reason, String sqlState, Throwable c
      * @param vendorCode
      * @param cause
      */
-    public SparkThriftServerSQLException(String reason, String sqlState, int vendorCode, Throwable cause) {
+    public SparkThriftServerSQLException(String reason,
+                                         String sqlState,
+                                         int vendorCode,
+                                         Throwable cause) {
         super(reason, sqlState, vendorCode, cause);
     }
 
@@ -197,7 +200,9 @@ public static Throwable toCause(List<String> details) {
         return toStackTrace(details, null, 0);
     }
 
-    private static Throwable toStackTrace(List<String> details, StackTraceElement[] parent, int index) {
+    private static Throwable toStackTrace(List<String> details,
+                                          StackTraceElement[] parent,
+                                          int index) {
         String detail = details.get(index++);
         if (!detail.startsWith("*")) {
             return null;  // should not be happened. ignore remaining
@@ -241,7 +246,8 @@ private static Throwable toStackTrace(List<String> details, StackTraceElement[]
 
     private static Throwable newInstance(String className, String message) {
         try {
-            return (Throwable)Class.forName(className).getConstructor(String.class).newInstance(message);
+            return (Throwable)Class.forName(className)
+                    .getConstructor(String.class).newInstance(message);
         } catch (Exception e) {
             return new RuntimeException(className + ":" + message);
         }

sql/thriftserver/src/main/java/org/apache/spark/sql/thriftserver/cli/operation/LogDivertAppender.java

@@ -130,7 +130,8 @@ private void setLayout(boolean isVerbose, Layout lo) {
         if (isVerbose) {
             if (lo == null) {
                 lo = CLIServiceUtils.verboseLayout;
-                LOG.info("Cannot find a Layout from a ConsoleAppender. Using default Layout pattern.");
+                LOG.info("Cannot find a Layout from a ConsoleAppender. " +
+                        "Using default Layout pattern.");
             }
         } else {
             lo = CLIServiceUtils.nonVerboseLayout;
@@ -171,7 +172,8 @@ public void doAppend(LoggingEvent event) {
 
         // Set current layout depending on the verbose/non-verbose mode.
         if (log != null) {
-            boolean isCurrModeVerbose = (log.getOpLoggingLevel() == OperationLog.LoggingLevel.VERBOSE);
+            boolean isCurrModeVerbose =
+                    (log.getOpLoggingLevel() == OperationLog.LoggingLevel.VERBOSE);
 
             // If there is a logging level change from verbose->non-verbose or vice-versa since
             // the last subAppend call, change the layout to preserve consistency.

sql/thriftserver/src/main/scala/org/apache/spark/sql/thriftserver/auth/PlainSaslHelper.scala

@@ -102,7 +102,7 @@ private class PlainServerCallbackHandler @throws[AuthenticationException]
       }
     }
     val provider = AuthenticationProviderFactory.getAuthenticationProvider(authMethod)
-    provider.Authenticate(username, password)
+    provider.authenticate(username, password)
     if (ac != null) {
       ac.setAuthorized(true)
     }

sql/thriftserver/src/main/scala/org/apache/spark/sql/thriftserver/cli/thrift/ThriftHttpServlet.scala

@@ -289,7 +289,7 @@ private[thriftserver] class ThriftHttpServlet(processor: TProcessor,
           AuthMethods.getValidAuthMethod(authType)
         val provider: PasswdAuthenticationProvider =
           AuthenticationProviderFactory.getAuthenticationProvider(authMethod)
-        provider.Authenticate(userName, getPassword(request, authType))
+        provider.authenticate(userName, getPassword(request, authType))
       } catch {
         case e: Exception =>
           throw new HttpAuthenticationException(e)
@@ -359,7 +359,7 @@ private[thriftserver] class ThriftHttpServlet(processor: TProcessor,
         val serviceTicketBase64: String = getAuthHeader(request, authType)
         val inToken: Array[Byte] = Base64.decodeBase64(serviceTicketBase64.getBytes)
         gssContext.acceptSecContext(inToken, 0, inToken.length)
-        // Authenticate or deny based on its context completion
+        // authenticate or deny based on its context completion
         if (!gssContext.isEstablished) {
           throw new HttpAuthenticationException("Kerberos authentication failed: " +
             "unable to establish context with the service ticket " + "provided by the client.")