Skip to content

Conversation

@yinxusen
Copy link
Contributor

@WangTaoTheTonic
Copy link
Contributor

@SparkQA
Copy link

SparkQA commented Apr 29, 2015

Test build #31252 has finished for PR 5769 at commit 43235f4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Repartition(numPartitions: Int, shuffle: Boolean, child: LogicalPlan)
    • case class RepartitionByExpression(partitionExpressions: Seq[Expression], child: LogicalPlan)
    • case class Repartition(numPartitions: Int, shuffle: Boolean, child: SparkPlan)
  • This patch does not change any dependencies.

@yinxusen
Copy link
Contributor Author

@WangTaoTheTonic Yes, there are three "threshhold" in spark package in total after I grep it. @mengxr If I modify the file in sql package, can you merge it?

@asfgit asfgit closed this in c0c0ba6 Apr 29, 2015
@mengxr
Copy link
Contributor

mengxr commented Apr 29, 2015

Yes, but let's do that separately. I've merged this into master. I didn't backport it because this is in a private method.

jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
mengxr

Author: Xusen Yin <[email protected]>

Closes apache#5769 from yinxusen/patch-1 and squashes the following commits:

43235f4 [Xusen Yin] Update PearsonCorrelation.scala
f7287ee [Xusen Yin] Fix a typo of "threshold"
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
mengxr

Author: Xusen Yin <[email protected]>

Closes apache#5769 from yinxusen/patch-1 and squashes the following commits:

43235f4 [Xusen Yin] Update PearsonCorrelation.scala
f7287ee [Xusen Yin] Fix a typo of "threshold"
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
mengxr

Author: Xusen Yin <[email protected]>

Closes apache#5769 from yinxusen/patch-1 and squashes the following commits:

43235f4 [Xusen Yin] Update PearsonCorrelation.scala
f7287ee [Xusen Yin] Fix a typo of "threshold"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants