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

    https://github.com/apache/spark/pull/21652#discussion_r201358555
  
    --- Diff: 
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesSuite.scala
 ---
    @@ -31,19 +31,18 @@ import scala.collection.JavaConverters._
     import org.apache.spark.SparkFunSuite
     import 
org.apache.spark.deploy.k8s.integrationtest.backend.{IntegrationTestBackend, 
IntegrationTestBackendFactory}
     import org.apache.spark.deploy.k8s.integrationtest.config._
    -import org.apache.spark.launcher.SparkLauncher
     
     private[spark] class KubernetesSuite extends SparkFunSuite
    -  with BeforeAndAfterAll with BeforeAndAfter {
    +  with BeforeAndAfterAll with BeforeAndAfter with BasicTestsSuite with 
SecretsTestsSuite {
     
       import KubernetesSuite._
     
       private var testBackend: IntegrationTestBackend = _
       private var sparkHomeDir: Path = _
    -  private var kubernetesTestComponents: KubernetesTestComponents = _
    -  private var sparkAppConf: SparkAppConf = _
    +  protected var kubernetesTestComponents: KubernetesTestComponents = _
    --- End diff --
    
    Just want to get them available at the derived classes.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to