Address review comments and fix styles

kimoonkim · kimoonkim · commit 0141c0a54d52 · 2017-08-01T15:35:26.000-07:00
diff --git a/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/kubernetes/submit/submitsteps/DriverHadoopCredentialsStep.scala b/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/kubernetes/submit/submitsteps/DriverHadoopCredentialsStep.scala
@@ -29,11 +29,10 @@ private[spark] class DriverHadoopCredentialsStep(submissionSparkConf: SparkConf)
     val podWithMountedHadoopToken = HadoopSecretUtil.configurePod(maybeMountedHadoopSecret,
       driverSpec.driverPod)
     val containerWithMountedHadoopToken = HadoopSecretUtil.configureContainer(
-      maybeMountedHadoopSecret, driverSpec.driverContainer)
+      maybeMountedHadoopSecret,
+      driverSpec.driverContainer)
     driverSpec.copy(
       driverPod = podWithMountedHadoopToken,
-      otherKubernetesResources = driverSpec.otherKubernetesResources,
-      driverSparkConf = driverSpec.driverSparkConf,
       driverContainer = containerWithMountedHadoopToken)
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -29,11 +29,10 @@ private[spark] class DriverHadoopCredentialsStep(submissionSparkConf: SparkConf)`
`29`	`29`	`val podWithMountedHadoopToken = HadoopSecretUtil.configurePod(maybeMountedHadoopSecret,`
`30`	`30`	`driverSpec.driverPod)`
`31`	`31`	`val containerWithMountedHadoopToken = HadoopSecretUtil.configureContainer(`
`32`		`- maybeMountedHadoopSecret, driverSpec.driverContainer)`
	`32`	`+ maybeMountedHadoopSecret,`
	`33`	`+ driverSpec.driverContainer)`
`33`	`34`	`driverSpec.copy(`
`34`	`35`	`driverPod = podWithMountedHadoopToken,`
`35`		`- otherKubernetesResources = driverSpec.otherKubernetesResources,`
`36`		`- driverSparkConf = driverSpec.driverSparkConf,`
`37`	`36`	`driverContainer = containerWithMountedHadoopToken)`
`38`	`37`	`}`
`39`	`38`	`}`