You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/12/17 00:54:27 UTC

[GitHub] [spark] dongjoon-hyun commented on pull request #38357: [SPARK-40887][K8S] Allow Spark on K8s to integrate w/ Log Service

dongjoon-hyun commented on PR #38357:
URL: https://github.com/apache/spark/pull/38357#issuecomment-1355886073

   Hi, @pan3793 . I'm interested in this PR for Apache Spark 3.4.0. To do that we need to split this PR in order to discuss and test more. I proposed you the followings first. Please let me know if you can revise.
   - Spin off pod env variable contributions (`SPARK_DRIVER_POD_NAME`).
   - Reduce the code change (e.g. Avoiding a whole new class like `KubernetesCoarseGrainedExecutorBackend`)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org