juliuszsompolski commented on code in PR #55917:
URL: https://github.com/apache/spark/pull/55917#discussion_r3638721023


##########
sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SparkConnectService.scala:
##########
@@ -314,6 +314,7 @@ class SparkConnectService(debug: Boolean) extends 
AsyncService with BindableServ
  */
 object SparkConnectService extends Logging {
 
+  private[connect] val SPARK_CONNECT_SESSION_USER_KEY: String = 
"spark.connect.session.user"

Review Comment:
   Could we use CurrentUserContext instead of SparkContext local property, the 
way Hive Thriftserver does?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to