Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r215435777
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesExecutorBuilder.scala
---
@@ -17,8 +17,8 @@
package org.apache.spark.scheduler.cluster.k8s
import org.apache.spark.deploy.k8s.{KubernetesConf,
KubernetesExecutorSpecificConf, KubernetesRoleSpecificConf, SparkPod}
-import org.apache.spark.deploy.k8s.features._
-import org.apache.spark.deploy.k8s.features.{BasicExecutorFeatureStep,
EnvSecretsFeatureStep, LocalDirsFeatureStep, MountSecretsFeatureStep}
+import org.apache.spark.deploy.k8s.Constants._
+import org.apache.spark.deploy.k8s.features.{BasicExecutorFeatureStep,
EnvSecretsFeatureStep, HadoopConfExecutorFeatureStep,
HadoopSparkUserExecutorFeatureStep, KerberosConfExecutorFeatureStep,
KubernetesFeatureConfigStep, LocalDirsFeatureStep, MountSecretsFeatureStep,
MountVolumesFeatureStep}
--- End diff --
Same: if the import list is getting this noisy, just import everything.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
