Github user tgravescs commented on the pull request:
https://github.com/apache/incubator-spark/pull/332#issuecomment-35807187
I think I have addressed all the comments and questions. If I missed
something please let me know.
Note I thought about moving the security files
(SparkSaslClient/SparkSaslServer/SecurityManager) into a security
sub-directory. Let me know what you think about that.
Also if there is preferred or more optimal way to do the
SparkSaslDigestCallbackHandler and SparkSaslClientCallbackHandler rather then
having the private nested class let me know.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. To do so, please top-post your response.
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
infrastruct...@apache.org or file a JIRA ticket with INFRA.
---
