Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Jul 11, 2018

What changes were proposed in this pull request?

Use longs in calculating min hash to avoid bias due to int overflow.

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Jul 11, 2018

Test build #92877 has finished for PR 21750 at commit aed50d7.

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

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Test build #92942 has finished for PR 21750 at commit edaed32.

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

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92953 has finished for PR 21750 at commit 55f70ee.

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

@srowen
Copy link
Member Author

srowen commented Jul 14, 2018

Merged to master

@asfgit asfgit closed this in 8aceb96 Jul 14, 2018
@srowen srowen deleted the SPARK-24754 branch July 16, 2018 14:53
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.

2 participants