Github user vanzin commented on the issue:

    https://github.com/apache/spark/pull/15981
  
    > Since it sends the key over the wire, I think it's better to throw an 
exception if securityManager.isSaslEncryptionEnabled() is false.
    
    We don't do that now, and there's a whole lot of things that need to 
configured correctly for things to be really secure (like HDFS and YARN 
encryption). It would also make unit tests slightly more complicated (or at 
least their set up). I think it's fine to just have a warning here.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to