GitHub user zsxwing opened a pull request:

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

    [SPARK-13747][SQL]Fix concurrent executions in ForkJoinPool for SQL

    ## What changes were proposed in this pull request?
    
    Calling `Await.result` will allow other tasks to be run on the same thread 
when using ForkJoinPool. However, SQL uses a `ThreadLocal` execution id to 
trace Spark jobs launched by a query, which doesn't work perfectly in 
ForkJoinPool.
    
    This PR just uses `Awaitable.result` instead to  prevent ForkJoinPool from 
running other tasks in the current waiting thread.
    
    ## How was this patch tested?
    
    Jenkins

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

    $ git pull https://github.com/zsxwing/spark SPARK-13747

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

    https://github.com/apache/spark/pull/15520.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 #15520
    
----
commit 54078ce1e33a05c740840e74f37834934f085d79
Author: Shixiong Zhu <shixi...@databricks.com>
Date:   2016-10-17T20:57:10Z

    Fix concurrent executions in ForkJoinPool for SQL

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to