Skip to content

Conversation

@yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Apr 23, 2020

What changes were proposed in this pull request?

_FUNC_ is used in note() of ExpressionDescription since #28248, it can be more cases later, we should replace it with function name for documentation

Why are the changes needed?

doc fix

Does this PR introduce any user-facing change?

no

How was this patch tested?

pass Jenkins, and verify locally with Jekyll serve

@HyukjinKwon
Copy link
Member

Documentation build passed in the Github Actions. I am going to merge this.

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.0.

HyukjinKwon pushed a commit that referenced this pull request Apr 23, 2020
…name in the note field of expression info

### What changes were proposed in this pull request?

\_FUNC\_ is used in note() of `ExpressionDescription` since #28248, it can be more cases later, we should replace it with function name for documentation

### Why are the changes needed?

doc fix

### Does this PR introduce any user-facing change?

no

### How was this patch tested?

pass Jenkins, and verify locally with Jekyll serve

Closes #28305 from yaooqinn/SPARK-31474-F.

Authored-by: Kent Yao <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 3b57921)
Signed-off-by: HyukjinKwon <[email protected]>
@SparkQA
Copy link

SparkQA commented Apr 23, 2020

Test build #121648 has finished for PR 28305 at commit 2a3d5ce.

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

Sign up for free to 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