Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Jun 12, 2017

What changes were proposed in this pull request?

This patch fixes a bug that can cause NullPointerException in LikeSimplification, when the pattern for like is null.

How was this patch tested?

Added a new unit test case in LikeSimplificationSuite.

@SparkQA
Copy link

SparkQA commented Jun 12, 2017

Test build #77923 has finished for PR 18273 at commit dc25912.

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

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

@gatorsmile
Copy link
Member

Thanks! Merging to master/2.2

asfgit pushed a commit that referenced this pull request Jun 12, 2017
## What changes were proposed in this pull request?
This patch fixes a bug that can cause NullPointerException in LikeSimplification, when the pattern for like is null.

## How was this patch tested?
Added a new unit test case in LikeSimplificationSuite.

Author: Reynold Xin <[email protected]>

Closes #18273 from rxin/SPARK-21059.

(cherry picked from commit b1436c7)
Signed-off-by: Xiao Li <[email protected]>
@asfgit asfgit closed this in b1436c7 Jun 12, 2017
dataknocker pushed a commit to dataknocker/spark that referenced this pull request Jun 16, 2017
## What changes were proposed in this pull request?
This patch fixes a bug that can cause NullPointerException in LikeSimplification, when the pattern for like is null.

## How was this patch tested?
Added a new unit test case in LikeSimplificationSuite.

Author: Reynold Xin <[email protected]>

Closes apache#18273 from rxin/SPARK-21059.
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