HyukjinKwon commented on code in PR #50466: URL: https://github.com/apache/spark/pull/50466#discussion_r2027863330
########## core/src/main/scala/org/apache/spark/security/SocketAuthHelper.scala: ########## @@ -80,7 +88,9 @@ private[spark] class SocketAuthHelper(val conf: SparkConf) { * @param s The socket connected to the server. * @throws IllegalArgumentException If authentication fails. */ - def authToServer(s: Socket): Unit = { + def authToServer(socket: SocketChannel): Unit = { + if (isUnixDomainSock) return Review Comment: We stop this auth since it's covered by file permissions. -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org