[GitHub] [spark] vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path

2019-12-16 Thread GitBox
vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path URL: https://github.com/apache/spark/pull/26493#issuecomment-566176527 Only the flaky client deps test failed, so looks good. Merging to master.

[GitHub] [spark] vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path

2019-12-13 Thread GitBox
vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path URL: https://github.com/apache/spark/pull/26493#issuecomment-565610883 retest this please This is an automated message

[GitHub] [spark] vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path

2019-11-15 Thread GitBox
vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path URL: https://github.com/apache/spark/pull/26493#issuecomment-554545257 Then, as I asked, could you add that to the docs?

[GitHub] [spark] vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path

2019-11-15 Thread GitBox
vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path URL: https://github.com/apache/spark/pull/26493#issuecomment-554544094 During what setup? I'm familiar with the docs, thanks. The docs say to put that in your `spark-env.sh`.

[GitHub] [spark] vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path

2019-11-15 Thread GitBox
vanzin commented on issue #26493: [SPARK-29574][K8S] Add SPARK_DIST_CLASSPATH to the executor class path URL: https://github.com/apache/spark/pull/26493#issuecomment-554526940 I don't see any code in the k8s backend that's actually propagating `SPARK_DIST_CLASSPATH` to the pods. That's