Skip to content

[SPARK-34856][FOLLOWUP][SQL] Remove dead code from AnsiCast.typeCheckFailureMessage - #32004

Closed
gengliangwang wants to merge 1 commit into
apache:masterfrom
gengliangwang:SPARK-34856-followup
Closed

[SPARK-34856][FOLLOWUP][SQL] Remove dead code from AnsiCast.typeCheckFailureMessage#32004
gengliangwang wants to merge 1 commit into
apache:masterfrom
gengliangwang:SPARK-34856-followup

Conversation

@gengliangwang

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

After #31954, Array type is allowed to be cast as String type. So the customized conversion failure message branch from AnsiCast.typeCheckFailureMessage won't be reached anymore.
This PR is to remove the dead code.

Why are the changes needed?

Code clean up.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Just removing dead code.

@SparkQA

Copy link
Copy Markdown

Kubernetes integration test unable to build dist.

exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41300/

@gengliangwang

Copy link
Copy Markdown
MemberAuthor

retest this please.

@gengliangwang

Copy link
Copy Markdown
MemberAuthor

^^^ I haven't sent this jenkins command for months...Have to do it since GA is slow now.

@MaxGekkMaxGekk 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.

LGTM, I wonder don't we have any test for the "dead" code?

@gengliangwang

Copy link
Copy Markdown
MemberAuthor

I wonder don't we have any test for the "dead" code?

@MaxGekk The error message of casting array type to string is not checked in #30260. Sorry about that.

@SparkQA

Copy link
Copy Markdown

Kubernetes integration test unable to build dist.

exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41310/

@SparkQA

Copy link
Copy Markdown

Test build #136718 has finished for PR 32004 at commit 93fea67.

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

@srowen

Copy link
Copy Markdown
Member

Merged to master

@SparkQA

Copy link
Copy Markdown

Test build #136729 has finished for PR 32004 at commit 93fea67.

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

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.

8 participants

@gengliangwang@SparkQA@srowen@maropu@MaxGekk@cloud-fan@HyukjinKwon@dongjoon-hyun