Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-10422] [SQL] String column in InMemoryColumnarCache needs to override clone method #8578

Closed
wants to merge 3 commits into from

Conversation

yhuai
Copy link
Contributor

@yhuai yhuai commented Sep 2, 2015

@bignoggins
Copy link

👍

@SparkQA
Copy link

SparkQA commented Sep 2, 2015

Test build #41944 has finished for PR 8578 at commit 1469cb0.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

ctx.range(1, 30000).selectExpr("id % 500 as id").rdd.map(id => Tuple1(s"str_$id")).toDF("i")
val cached = df.cache()
// count triggers the caching action. It should not throw.
cached.count()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we check the results?


test("SPARK-10422: String column in InMemoryColumnarCache needs to override clone method") {
val df =
ctx.range(1, 100).selectExpr("id % 10 as id").rdd.map(id => Tuple1(s"str_$id")).toDF("i")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reduced the number of rows to make this test faster. I also checked that it did fail without the fix.

@SparkQA
Copy link

SparkQA commented Sep 3, 2015

Test build #1715 has finished for PR 8578 at commit 1469cb0.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 3, 2015

Test build #41952 has finished for PR 8578 at commit 2905fd5.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 3, 2015

Test build #1716 has finished for PR 8578 at commit 2905fd5.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@davies
Copy link
Contributor

davies commented Sep 3, 2015

LGTM, merging this into master 1.5

asfgit pushed a commit that referenced this pull request Sep 3, 2015
…verride clone method

https://issues.apache.org/jira/browse/SPARK-10422

Author: Yin Huai <yhuai@databricks.com>

Closes #8578 from yhuai/SPARK-10422.

(cherry picked from commit 03f3e91)
Signed-off-by: Davies Liu <davies.liu@gmail.com>
@asfgit asfgit closed this in 03f3e91 Sep 3, 2015
tianyi added a commit to growingio/spark that referenced this pull request Sep 18, 2015
ashangit pushed a commit to ashangit/spark that referenced this pull request Oct 19, 2016
…verride clone method

https://issues.apache.org/jira/browse/SPARK-10422

Author: Yin Huai <yhuai@databricks.com>

Closes apache#8578 from yhuai/SPARK-10422.

(cherry picked from commit 03f3e91)
Signed-off-by: Davies Liu <davies.liu@gmail.com>
(cherry picked from commit 2fce5d8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants