Address comments

Author: aarondav

core/src/main/scala/org/apache/spark/deploy/worker/WorkerShuffleService.scala renamed to core/src/main/scala/org/apache/spark/deploy/worker/StandaloneWorkerShuffleService.scala

@@ -32,7 +32,8 @@ import org.apache.spark.network.shuffle.ExternalShuffleBlockHandler
  * Optionally requires SASL authentication in order to read. See [[SecurityManager]].
  */
 private[worker]
-class WorkerShuffleService(sparkConf: SparkConf, securityManager: SecurityManager) extends Logging {
+class StandaloneWorkerShuffleService(sparkConf: SparkConf, securityManager: SecurityManager)
+  extends Logging {
 
   private val enabled = sparkConf.getBoolean("spark.shuffle.service.enabled", false)
   private val port = sparkConf.getInt("spark.shuffle.service.port", 7337)

core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala

@@ -112,7 +112,7 @@ private[spark] class Worker(
   val finishedDrivers = new HashMap[String, DriverRunner]
 
   // The shuffle service is not actually started unless configured.
-  var shuffleService = new WorkerShuffleService(conf, securityMgr)
+  val shuffleService = new StandaloneWorkerShuffleService(conf, securityMgr)
 
   val publicAddress = {
     val envVar = System.getenv("SPARK_PUBLIC_DNS")