Github user andrewor14 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3233#discussion_r24306601
  
    --- Diff: 
yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala ---
    @@ -17,23 +17,30 @@
     
     package org.apache.spark.deploy.yarn
     
    -import java.io.File
    +import java.io.{File, FileOutputStream, OutputStreamWriter}
    +import java.util.Properties
     import java.util.concurrent.TimeUnit
     
     import scala.collection.JavaConversions._
     import scala.collection.mutable
     
    -import com.google.common.base.Charsets
    +import com.google.common.base.Charsets.UTF_8
    +import com.google.common.io.ByteStreams
     import com.google.common.io.Files
     import org.apache.hadoop.yarn.conf.YarnConfiguration
     import org.apache.hadoop.yarn.server.MiniYARNCluster
     import org.scalatest.{BeforeAndAfterAll, FunSuite, Matchers}
     
    -import org.apache.spark.{Logging, SparkConf, SparkContext, SparkException}
    +import org.apache.spark.{Logging, SparkConf, SparkContext, SparkException, 
TestUtils}
     import org.apache.spark.scheduler.cluster.ExecutorInfo
     import org.apache.spark.scheduler.{SparkListener, 
SparkListenerExecutorAdded}
     import org.apache.spark.util.Utils
     
    +/**
    + * Integration tests for Yarn; these tests use a mini Yarn cluster to run 
Spark-on-Yarn
    --- End diff --
    
    I think elsewhere we use YARN instead of Yarn


---
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