odraese commented on a change in pull request #648: HIVE-21783: Accept Hive 
connections from the same domain without authentication.
URL: https://github.com/apache/hive/pull/648#discussion_r288186125
 
 

 ##########
 File path: service/src/java/org/apache/hive/service/auth/PlainSaslHelper.java
 ##########
 @@ -65,6 +70,15 @@ public static TTransportFactory 
getPlainTransportFactory(String authTypeStr)
     return saslFactory;
   }
 
+  public static TTransportFactory 
getDualPlainTransportFactory(TTransportFactory otherTrans,
+                                                               String hiveHost)
+          throws LoginException {
+    LOG.info("Created additional transport factory for skipping authentication 
when client " +
 
 Review comment:
   Maybe we should perform a validation of the input parameters (not null, not 
empty host string) here?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to