GitHub user liyinan926 opened a pull request:

    https://github.com/apache/spark/pull/19972

    [SPARK-22778][Kubernetes] Added the missing service metadata for 
KubernetesClusterManager

    ## What changes were proposed in this pull request?
    
    This PR added the missing service metadata for `KubernetesClusterManager`. 
Without the metadata, the service loader couldn't load 
`KubernetesClusterManager`, and caused the driver to fail to create a 
`ExternalClusterManager`, as being reported in SPARK-22778. The PR also changed 
the `k8s:` prefix used to `k8s://`, which is what existing Spark on k8s users 
are familiar and used to. 
    
    ## How was this patch tested?
    
    Manual testing verified that the fix resolved the issue in SPARK-22778.
    
    /cc k8s-big-data SIG members & contributors: @mccheah @foxish @ash211 
@ssuchter @varunkatta @kimoonkim @erikerlandson @tnachen @ifilonenko @liyinan926
    /cc @vanzin @felixcheung @jiangxb1987

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/liyinan926/spark-k8s fix-22778

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/19972.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #19972
    
----
commit 48c4b0a6f31b3d05151219f55f8ac545a8236102
Author: Yinan Li <[email protected]>
Date:   2017-12-13T21:05:41Z

    [SPARK-22778][Kubernetes] Added the missing service metadata for 
KubernetesClusterManager

----


---

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

Reply via email to