Skip to content

[SPARK-18411] [SQL] Add Argument Types and Test Cases for String Functions [WIP] - #15850

Closed
gatorsmile wants to merge 3 commits into
apache:masterfrom
gatorsmile:addArgument4StringExpressions
Closed

[SPARK-18411] [SQL] Add Argument Types and Test Cases for String Functions [WIP]#15850
gatorsmile wants to merge 3 commits into
apache:masterfrom
gatorsmile:addArgument4StringExpressions

Conversation

@gatorsmile

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Add argument types and test cases into the extended descriptions of string functions.

How was this patch tested?

Added test cases to verify the added argument types.

@SparkQA

Copy link
Copy Markdown

Test build #68511 has finished for PR 15850 at commit c02b10d.

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

@SparkQA

Copy link
Copy Markdown

Test build #68581 has finished for PR 15850 at commit 1ba3d67.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

Hi @gatorsmile, is this still WIP?

@HyukjinKwonHyukjinKwon mentioned this pull request May 17, 2017
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?
This PR proposes to close PRs ...
- inactive to the review comments more than a month
- WIP and inactive more than a month
- with Jenkins build failure but inactive more than a month
- suggested to be closed and no comment against that
- obviously looking inappropriate (e.g., Branch 0.5)
To make sure, I left a comment for each PR about a week ago and I could not have a response back from the author in these PRs below:
Closesapache#11129Closesapache#12085Closesapache#12162Closesapache#12419Closesapache#12420Closesapache#12491Closesapache#13762Closesapache#13837Closesapache#13851Closesapache#13881Closesapache#13891Closesapache#13959Closesapache#14091Closesapache#14481Closesapache#14547Closesapache#14557Closesapache#14686Closesapache#15594Closesapache#15652Closesapache#15850Closesapache#15914Closesapache#15918Closesapache#16285Closesapache#16389Closesapache#16652Closesapache#16743Closesapache#16893Closesapache#16975Closesapache#17001Closesapache#17088Closesapache#17119Closesapache#17272Closesapache#17971
Added:
Closesapache#17778Closesapache#17303Closesapache#17872
## How was this patch tested?
N/A
Author: hyukjinkwon <gurwls223@gmail.com>
Closesapache#18017 from HyukjinKwon/close-inactive-prs.
Sign up for freeto 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

@gatorsmile@SparkQA@HyukjinKwon