Thanks for looking at this PR mridul... I was going to cc you


On Thu, Feb 6, 2014 at 2:47 PM, Mridul Muralidharan <
notifications@github.com> wrote:

> In core/src/main/java/org/apache/spark/SparkSaslServer.java:
>
> > +import java.io.IOException;
> > +
> > +/**
> > + * Encapsulates SASL server logic for Server
> > + */
> > +public class SparkSaslServer {
> > +  /** Logger */
> > +  private static Logger LOG = LoggerFactory.getLogger("SparkSaslServer.class");
> > +
> > +  /**
> > +   * Actual SASL work done by this object from javax.security.sasl.
> > +   * Initialized below in constructor.
> > +   */
> > +  private SaslServer saslServer;
> > +
> > +  public static final String SASL_DEFAULT_REALM = "default";
>
> Not super important, but we might want to change this to something else ?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/apache/incubator-spark/pull/332/files#r9522830>
> .
>