Skip to content

Commit e00ebd5

Browse files
committed
[SPARK-26482][K8S][TEST][FOLLOWUP] Fix compile failure
## What changes were proposed in this pull request? This fixes K8S integration test compilation failure introduced by #23423 . ```scala $ build/sbt -Pkubernetes-integration-tests test:package ... [error] /Users/dongjoon/APACHE/spark/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesTestComponents.scala:71: type mismatch; [error] found : org.apache.spark.internal.config.OptionalConfigEntry[Boolean] [error] required: String [error] .set(IS_TESTING, false) [error] ^ [error] /Users/dongjoon/APACHE/spark/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesTestComponents.scala:71: type mismatch; [error] found : Boolean(false) [error] required: String [error] .set(IS_TESTING, false) [error] ^ [error] two errors found ``` ## How was this patch tested? Pass the K8S integration test. Closes #23527 from dongjoon-hyun/SPARK-26482. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 19e17ac commit e00ebd5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesTestComponents.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,8 @@ private[spark] class KubernetesTestComponents(defaultClient: DefaultKubernetesCl
6868
.set("spark.executor.cores", "1")
6969
.set("spark.executors.instances", "1")
7070
.set("spark.app.name", "spark-test-app")
71-
.set(IS_TESTING, false)
72-
.set(UI_ENABLED, true)
71+
.set(IS_TESTING.key, "false")
72+
.set(UI_ENABLED.key, "true")
7373
.set("spark.kubernetes.submission.waitAppCompletion", "false")
7474
.set("spark.kubernetes.authenticate.driver.serviceAccountName", serviceAccountName)
7575
}

0 commit comments

Comments
 (0)