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

Reply via email to