You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by breakdawn <gi...@git.apache.org> on 2016/07/29 17:51:45 UTC

[GitHub] spark pull request #14404: [SPARK-16664][SQL] Fix persist call on Data frame...

GitHub user breakdawn opened a pull request:

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

    [SPARK-16664][SQL] Fix persist call on Data frames with more than 200\u2026

    ## What changes were proposed in this pull request?
    
    Cherry-pick from d1d5069aa3744d46abd3889abab5f15e9067382a and fix the test case
    
    
    ## How was this patch tested?
    
    Test updated

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

    $ git pull https://github.com/breakdawn/spark branch-1.6

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

    https://github.com/apache/spark/pull/14404.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 #14404
    
----
commit 555b67f398876c36441a5d32fec0f39d38cc5222
Author: Wesley Tang <ta...@mininglamp.com>
Date:   2016-07-29T11:26:05Z

    [SPARK-16664][SQL] Fix persist call on Data frames with more than 200\u2026
    
    f12f11e578169b47e3f8b18b299948c0670ba585 introduced this bug, missed foreach as map
    
    Cherry-pick from d1d5069aa3744d46abd3889abab5f15e9067382a and update test case

----


---
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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    Great, thank you @breakdawn . It looks good as a re-submit of the original PR, and passes, so I'm merging.


---
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 #14404: [SPARK-16664][SQL] Fix persist call on Data frame...

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

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


---
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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    **[Test build #63008 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63008/consoleFull)** for PR 14404 at commit [`555b67f`](https://github.com/apache/spark/commit/555b67f398876c36441a5d32fec0f39d38cc5222).
     * 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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    You can close this PR now; we can't auto-close it and neither can the bot in this situation.


---
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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    Can you fix the title?



---
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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63008/
    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 #14404: [SPARK-16664][SQL] Fix persist call on Data frames with ...

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

    https://github.com/apache/spark/pull/14404
  
    **[Test build #63008 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63008/consoleFull)** for PR 14404 at commit [`555b67f`](https://github.com/apache/spark/commit/555b67f398876c36441a5d32fec0f39d38cc5222).


---
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