Skip to content

[SPARK-32018][SQL][2.4] UnsafeRow.setDecimal should set null with overflowed value - #29141

Closed
cloud-fan wants to merge 1 commit into
apache:branch-2.4from
cloud-fan:backport
Closed

[SPARK-32018][SQL][2.4] UnsafeRow.setDecimal should set null with overflowed value#29141
cloud-fan wants to merge 1 commit into
apache:branch-2.4from
cloud-fan:backport

Conversation

@cloud-fan

Copy link
Copy Markdown
Contributor

backport #29125

…rflowed value
partially backport apache#29026Closesapache#29125 from cloud-fan/backport.
Authored-by: Wenchen Fan <wenchen@databricks.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@cloud-fan

Copy link
Copy Markdown
ContributorAuthor

cc @dongjoon-hyun

@dongjoon-hyun

Copy link
Copy Markdown
Member

Thank you, @cloud-fan !

@SparkQA

Copy link
Copy Markdown

Test build #126030 has finished for PR 29141 at commit 3210002.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun

Copy link
Copy Markdown
Member

Retest this please.

@SparkQA

Copy link
Copy Markdown

Test build #126037 has finished for PR 29141 at commit 3210002.

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

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM. Merged to branch-2.4.

dongjoon-hyun pushed a commit that referenced this pull request Jul 17, 2020
…rflowed value
backport #29125Closes#29141 from cloud-fan/backport.
Authored-by: Wenchen Fan <wenchen@databricks.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@cloud-fan

cloud-fan commented Aug 17, 2020

Copy link
Copy Markdown
ContributorAuthor

I'm reverting it from 2.4, see #29450 (comment)

@dongjoon-hyun

Copy link
Copy Markdown
Member

Got it. I understand the situation. Thank you for the decision, @cloud-fan .

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@cloud-fan@dongjoon-hyun@SparkQA