hvanhovell commented on code in PR #49880:
URL: https://github.com/apache/spark/pull/49880#discussion_r1960776196
##########
python/pyspark/sql/connect/client/core.py:
##########
@@ -125,6 +125,7 @@ class ChannelBuilder:
PARAM_USER_ID = "user_id"
PARAM_USER_AGENT = "user_agent"
PARAM_SESSION_ID = "session_id"
+ CONNECT_LOCAL_AUTH_TOKEN_PARAM_NAME = "local_token"
Review Comment:
@Kimahriman are you suggesting to use a different header than
Authentication? If so, my issue with that is that folks will invariably use
this to 'secure' remote connections as well, and in that case having TLS might
not be such a bad thing.
We are planning to add UDS support in a follow-up. That should make this
less of an issue.
--
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]
