Skip to content

Conversation

@kiszk
Copy link
Member

@kiszk kiszk commented Jan 6, 2021

What changes were proposed in this pull request?

This PR is a follow-up of #31061. It fixes a typo in a document: Finctions -> Functions

Why are the changes needed?

Make the change better documented.

Does this PR introduce any user-facing change?

No

How was this patch tested?

N/A

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38333/

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38333/

@SparkQA
Copy link

SparkQA commented Jan 6, 2021

Test build #133745 has finished for PR 31069 at commit 9199300.

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

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @kiszk .
Merged to master/3.1.

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-34022][DOCS][FOLLOW-UP] Support latest mkdocs in SQL built-in function docs [SPARK-34022][DOCS][FOLLOW-UP] Fix typo in SQL built-in function docs Jan 6, 2021
dongjoon-hyun pushed a commit that referenced this pull request Jan 6, 2021
### What changes were proposed in this pull request?

This PR is a follow-up of #31061. It fixes a typo in a document: `Finctions` -> `Functions`

### Why are the changes needed?

Make the change better documented.

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

No

### How was this patch tested?

N/A

Closes #31069 from kiszk/SPARK-34022-followup.

Authored-by: Kazuaki Ishizaki <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit a0269bb)
Signed-off-by: Dongjoon Hyun <[email protected]>
@github-actions github-actions bot added the SQL label Jan 6, 2021
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