You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by squito <gi...@git.apache.org> on 2017/12/21 16:56:28 UTC

[GitHub] spark pull request #20047: [SPARK-22861][SQL] SQLAppStatusListener handles m...

GitHub user squito opened a pull request:

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

    [SPARK-22861][SQL] SQLAppStatusListener handles multi-job executions.

    When one execution has multiple jobs, we need to append to the set of
    stages, not replace them on every job.
    
    Added unit test and ran existing tests on jenkins

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

    $ git pull https://github.com/squito/spark SPARK-22861

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

    https://github.com/apache/spark/pull/20047.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 #20047
    
----
commit 29d184c1e39c8c9203ff1b5988e7f429680fce2d
Author: Imran Rashid <ir...@...>
Date:   2017-12-21T16:54:14Z

    [SPARK-22861][SQL] SQLAppStatusListener handles multi-job executions.
    
    When one execution has multiple jobs, we need to append to the set of
    stages, not replace them on every job.

----


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    Merging to master.


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    Merged build finished. Test FAILed.


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    **[Test build #85274 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85274/testReport)** for PR 20047 at commit [`28ee8a9`](https://github.com/apache/spark/commit/28ee8a9a92547bdb30a771c693b5bff2e2649b77).


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    **[Test build #85280 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85280/testReport)** for PR 20047 at commit [`89e1c6d`](https://github.com/apache/spark/commit/89e1c6d4cb75dfdadf3de768754fc92c985f4d73).


---

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


[GitHub] spark pull request #20047: [SPARK-22861][SQL] SQLAppStatusListener handles m...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    **[Test build #85274 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85274/testReport)** for PR 20047 at commit [`28ee8a9`](https://github.com/apache/spark/commit/28ee8a9a92547bdb30a771c693b5bff2e2649b77).
     * This patch **fails to build**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    **[Test build #85271 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85271/testReport)** for PR 20047 at commit [`29d184c`](https://github.com/apache/spark/commit/29d184c1e39c8c9203ff1b5988e7f429680fce2d).


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    @vanzin @carsonwang 


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

    https://github.com/apache/spark/pull/20047
  
    Looks good but you have to fix conflicts now.


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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


[GitHub] spark issue #20047: [SPARK-22861][SQL] SQLAppStatusListener handles multi-jo...

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

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


---

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