Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Aug 2, 2017

What changes were proposed in this pull request?

When the watermark is not a column of dropDuplicates, right now it will crash. This PR fixed this issue.

How was this patch tested?

The new unit test.

@zsxwing zsxwing changed the title [SPARK-21546] dropDuplicates should ignore watermark when it's not a key [SPARK-21546][SS] dropDuplicates should ignore watermark when it's not a key Aug 2, 2017
@marmbrus
Copy link
Contributor

marmbrus commented Aug 2, 2017

LGTM

@SparkQA
Copy link

SparkQA commented Aug 2, 2017

Test build #80172 has finished for PR 18822 at commit 2b34308.

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

@zsxwing
Copy link
Member Author

zsxwing commented Aug 2, 2017

Thanks! Merging to master and 2.2.

asfgit pushed a commit that referenced this pull request Aug 2, 2017
…t a key

## What changes were proposed in this pull request?

When the watermark is not a column of `dropDuplicates`, right now it will crash. This PR fixed this issue.

## How was this patch tested?

The new unit test.

Author: Shixiong Zhu <[email protected]>

Closes #18822 from zsxwing/SPARK-21546.

(cherry picked from commit 0d26b3a)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in 0d26b3a Aug 2, 2017
@zsxwing zsxwing deleted the SPARK-21546 branch August 2, 2017 21:18
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
…t a key

## What changes were proposed in this pull request?

When the watermark is not a column of `dropDuplicates`, right now it will crash. This PR fixed this issue.

## How was this patch tested?

The new unit test.

Author: Shixiong Zhu <[email protected]>

Closes apache#18822 from zsxwing/SPARK-21546.

(cherry picked from commit 0d26b3a)
Signed-off-by: Shixiong Zhu <[email protected]>
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