Hi Ye,

I think git blame shows me because I fixed the formatting in core/pom.xml,
but I don't actually know the original reason for setting SPARK_CLASSPATH
there.

Do the tests run OK if you take it out?

-Sandy


On Thu, Sep 25, 2014 at 1:59 AM, Ye Xianjin <advance...@gmail.com> wrote:

> hi, Sandy Ryza:
>      I believe It's you originally added the SPARK_CLASSPATH in
> core/pom.xml in the org.scalatest section. Does this still needed in 1.1?
>      I noticed this setting because when I looked into the unit-tests.log,
> It shows something below:
> > 14/09/24 23:57:19.246 WARN SparkConf:
> > SPARK_CLASSPATH was detected (set to 'null').
> > This is deprecated in Spark 1.0+.
> >
> > Please instead use:
> >  - ./spark-submit with --driver-class-path to augment the driver
> classpath
> >  - spark.executor.extraClassPath to augment the executor classpath
> >
> > 14/09/24 23:57:19.246 WARN SparkConf: Setting
> 'spark.executor.extraClassPath' to 'null' as a work-around.
> > 14/09/24 23:57:19.247 WARN SparkConf: Setting
> 'spark.driver.extraClassPath' to 'null' as a work-around.
>
> However I didn't set SPARK_CLASSPATH env variable. And looked into the
> SparkConf.scala, If user actually set extraClassPath,  the SparkConf will
> throw SparkException.
> --
> Ye Xianjin
> Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
>
>
> On Tuesday, September 23, 2014 at 12:56 AM, Ye Xianjin wrote:
>
> > Hi:
> >     I notice the scalatest-maven-plugin set SPARK_CLASSPATH environment
> variable for testing. But in the SparkConf.scala, this is deprecated in
> Spark 1.0+.
> >     So what this variable for? should we just remove this variable?
> >
> >
> > --
> > Ye Xianjin
> > Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> >
>
>

Reply via email to