Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Apr 30, 2016

What changes were proposed in this pull request?

This patch fixes a null handling bug in EqualNullSafe's code generation.

How was this patch tested?

Updated unit test so they would fail without the fix.

Closes #12628.

Arash Nabili and others added 3 commits April 22, 2016 16:54
@rxin
Copy link
Contributor Author

rxin commented Apr 30, 2016

cc @davies

@SparkQA
Copy link

SparkQA commented Apr 30, 2016

Test build #57396 has finished for PR 12799 at commit 761b9d7.

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

@davies
Copy link
Contributor

davies commented Apr 30, 2016

LGTM,
Merging this into master, thanks!

@asfgit asfgit closed this in 7945f9f Apr 30, 2016
asfgit pushed a commit that referenced this pull request Apr 30, 2016
This patch fixes a null handling bug in EqualNullSafe's code generation.

Updated unit test so they would fail without the fix.

Closes #12628.

Author: Reynold Xin <[email protected]>
Author: Arash Nabili <[email protected]>

Closes #12799 from rxin/equalnullsafe.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request May 3, 2016
This patch fixes a null handling bug in EqualNullSafe's code generation.

Updated unit test so they would fail without the fix.

Closes apache#12628.

Author: Reynold Xin <[email protected]>
Author: Arash Nabili <[email protected]>

Closes apache#12799 from rxin/equalnullsafe.

(cherry picked from commit 10d513f)
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