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

    https://github.com/apache/spark/pull/22298#discussion_r214408969
  
    --- Diff: 
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/PythonTestsSuite.scala
 ---
    @@ -72,12 +72,33 @@ private[spark] trait PythonTestsSuite { k8sSuite: 
KubernetesSuite =>
           isJVM = false,
           pyFiles = Some(PYSPARK_CONTAINER_TESTS))
       }
    +
    +  test("Run PySpark with memory customization", k8sTestTag) {
    +    sparkAppConf
    +      .set("spark.kubernetes.container.image", 
s"${getTestImageRepo}/spark-py:${getTestImageTag}")
    --- End diff --
    
    nit: Mildly confused why there's so many sets here (like the image, etc.) 
maybe make more sense in a shared test setup func?


---

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

Reply via email to