You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by vanzin <gi...@git.apache.org> on 2017/03/03 22:27:29 UTC

[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

GitHub user vanzin opened a pull request:

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

    [SPARK-19084][sql] Ensure context class loader is set when initializing Hive.

    A change in Hive 2.2 (most probably HIVE-13149) causes this code path to fail,
    since the call to "state.getConf.setClassLoader" does not actually change the
    context's class loader. Spark doesn't yet officially support Hive 2.2, but some
    distribution-specific metastore client libraries may have that change (as certain
    versions of CDH already do), and this also makes it easier to support 2.2 when it
    comes out.
    
    Tested with existing unit tests; we've also used this patch extensively with Hive
    metastore client jars containing the offending patch.


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

    $ git pull https://github.com/vanzin/spark SPARK-19804

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

    https://github.com/apache/spark/pull/17154.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 #17154
    
----
commit 324619620fff32b766068a0ea0f04aef67b2780e
Author: Marcelo Vanzin <va...@cloudera.com>
Date:   2017-03-03T19:27:41Z

    [SPARK-19084][sql] Ensure context class loader is set when initializing Hive.
    
    A change in Hive 2.2 (most probably HIVE-13149) causes this code path to fail,
    since the call to "state.getConf.setClassLoader" does not actually change the
    context's class loader. Spark doesn't yet officially support Hive 2.2, but some
    distribution-specific metastore client libraries may have that change (as certain
    versions of CDH already do), and this also makes it easier to support 2.2 when it
    comes out.
    
    Tested with existing unit tests; we've also used this patch extensively with Hive
    metastore client jars containing the offending patch.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73880/
    Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by vanzin <gi...@git.apache.org>.
Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17154#discussion_r104270472
  
    --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala ---
    @@ -268,16 +268,19 @@ private[hive] class HiveClientImpl(
        */
       def withHiveState[A](f: => A): A = retryLocked {
         val original = Thread.currentThread().getContextClassLoader
    -    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    -    Hive.set(client)
    +    val originalConfLoader = state.getConf.getClassLoader
         // The classloader in clientLoader could be changed after addJar, always use the latest
         // classloader
    +    Thread.currentThread().setContextClassLoader(clientLoader.classLoader)
         state.getConf.setClassLoader(clientLoader.classLoader)
    --- End diff --
    
    Yes; I'm not sure the second one is strictly necessary, but from a cleanliness p.o.v. it looks like the right thing to do.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73865 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73865/testReport)** for PR 17154 at commit [`3246196`](https://github.com/apache/spark/commit/324619620fff32b766068a0ea0f04aef67b2780e).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by vanzin <gi...@git.apache.org>.
Github user vanzin commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    @cloud-fan @gatorsmile 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by gatorsmile <gi...@git.apache.org>.
Github user gatorsmile commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    retest this please


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by cloud-fan <gi...@git.apache.org>.
Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17154#discussion_r104270414
  
    --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala ---
    @@ -268,16 +268,19 @@ private[hive] class HiveClientImpl(
        */
       def withHiveState[A](f: => A): A = retryLocked {
         val original = Thread.currentThread().getContextClassLoader
    -    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    -    Hive.set(client)
    +    val originalConfLoader = state.getConf.getClassLoader
         // The classloader in clientLoader could be changed after addJar, always use the latest
         // classloader
    +    Thread.currentThread().setContextClassLoader(clientLoader.classLoader)
         state.getConf.setClassLoader(clientLoader.classLoader)
    --- End diff --
    
    looks like we fixed 2 problems here: 1. set thread context class loader. 2. set class loader back for ` state.getConf`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by gatorsmile <gi...@git.apache.org>.
Github user gatorsmile commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Thanks! Merging to master. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by cloud-fan <gi...@git.apache.org>.
Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17154#discussion_r104270996
  
    --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala ---
    @@ -268,16 +268,19 @@ private[hive] class HiveClientImpl(
        */
       def withHiveState[A](f: => A): A = retryLocked {
         val original = Thread.currentThread().getContextClassLoader
    -    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    -    Hive.set(client)
    +    val originalConfLoader = state.getConf.getClassLoader
         // The classloader in clientLoader could be changed after addJar, always use the latest
         // classloader
    +    Thread.currentThread().setContextClassLoader(clientLoader.classLoader)
    --- End diff --
    
    shall we add some comments to say why we need to do this?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73878/
    Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Merged build finished. Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73865 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73865/testReport)** for PR 17154 at commit [`3246196`](https://github.com/apache/spark/commit/324619620fff32b766068a0ea0f04aef67b2780e).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73880 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73880/testReport)** for PR 17154 at commit [`aa2a66b`](https://github.com/apache/spark/commit/aa2a66b8973f0dc8e9b6b14a7b4112036df8d1ba).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by gatorsmile <gi...@git.apache.org>.
Github user gatorsmile commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    This is to ensure the testing pick up the latest changes we made to support Hive metastore 2.0


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Merged build finished. Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by cloud-fan <gi...@git.apache.org>.
Github user cloud-fan commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73880 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73880/testReport)** for PR 17154 at commit [`aa2a66b`](https://github.com/apache/spark/commit/aa2a66b8973f0dc8e9b6b14a7b4112036df8d1ba).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by vanzin <gi...@git.apache.org>.
Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17154#discussion_r104271357
  
    --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala ---
    @@ -268,16 +268,19 @@ private[hive] class HiveClientImpl(
        */
       def withHiveState[A](f: => A): A = retryLocked {
         val original = Thread.currentThread().getContextClassLoader
    -    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    -    Hive.set(client)
    +    val originalConfLoader = state.getConf.getClassLoader
         // The classloader in clientLoader could be changed after addJar, always use the latest
         // classloader
    +    Thread.currentThread().setContextClassLoader(clientLoader.classLoader)
    --- End diff --
    
    Sure.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Merged build finished. Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73878 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73878/testReport)** for PR 17154 at commit [`aa2a66b`](https://github.com/apache/spark/commit/aa2a66b8973f0dc8e9b6b14a7b4112036df8d1ba).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by BlueStalker <gi...@git.apache.org>.
Github user BlueStalker commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    @gatorsmile I want to understand the scope of this fix.
    Without the fix , it will have exception if spark trying to load the hive meta store 2.x version, correct?



---

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


[GitHub] spark pull request #17154: [SPARK-19084][sql] Ensure context class loader is...

Posted by cloud-fan <gi...@git.apache.org>.
Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17154#discussion_r104270279
  
    --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala ---
    @@ -268,16 +268,19 @@ private[hive] class HiveClientImpl(
        */
       def withHiveState[A](f: => A): A = retryLocked {
         val original = Thread.currentThread().getContextClassLoader
    -    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    -    Hive.set(client)
    +    val originalConfLoader = state.getConf.getClassLoader
         // The classloader in clientLoader could be changed after addJar, always use the latest
         // classloader
    +    Thread.currentThread().setContextClassLoader(clientLoader.classLoader)
         state.getConf.setClassLoader(clientLoader.classLoader)
    +    // Set the thread local metastore client to the client associated with this HiveClientImpl.
    +    Hive.set(client)
         // setCurrentSessionState will use the classLoader associated
         // with the HiveConf in `state` to override the context class loader of the current
         // thread.
         shim.setCurrentSessionState(state)
         val ret = try f finally {
    +      state.getConf.setClassLoader(originalConfLoader)
    --- End diff --
    
    why we set it back? where we change it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    **[Test build #73878 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73878/testReport)** for PR 17154 at commit [`aa2a66b`](https://github.com/apache/spark/commit/aa2a66b8973f0dc8e9b6b14a7b4112036df8d1ba).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #17154: [SPARK-19084][sql] Ensure context class loader is set wh...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/17154
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73865/
    Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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