Skip to content

[SPARK-10740][SQL] handle nondeterministic expressions correctly for set operations #8858

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

Closed
wants to merge 1 commit into from

Conversation

cloud-fan
Copy link
Contributor

@SparkQA
Copy link

SparkQA commented Sep 22, 2015

Test build #42794 has finished for PR 8858 at commit a1f41b5.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 22, 2015

Test build #42799 has finished for PR 8858 at commit 555cf3b.

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

def expected(df: DataFrame): Seq[Row] = {
df.rdd.collectPartitions().zipWithIndex.flatMap {
case (data, index) =>
val rng = new org.apache.spark.util.random.XORShiftRandom(7 + index)
Copy link
Contributor

Choose a reason for hiding this comment

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

It will be good to have a comment to say we need to match the behavior of RAND when generating expected results.

@yhuai
Copy link
Contributor

yhuai commented Sep 22, 2015

LGTM. Merging to master and branch 1.5.

asfgit pushed a commit that referenced this pull request Sep 22, 2015
… set operations

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

Author: Wenchen Fan <cloud0fan@163.com>

Closes #8858 from cloud-fan/non-deter.

(cherry picked from commit 5017c68)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 5017c68 Sep 22, 2015
ashangit pushed a commit to ashangit/spark that referenced this pull request Oct 19, 2016
… set operations

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

Author: Wenchen Fan <cloud0fan@163.com>

Closes apache#8858 from cloud-fan/non-deter.

(cherry picked from commit 5017c68)
Signed-off-by: Yin Huai <yhuai@databricks.com>
(cherry picked from commit 54334d3)
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.

3 participants