Skip to content

[MINOR][DOCS] Use ASCII characters when possible in PySpark documentation - #32048

Closed
HyukjinKwon wants to merge 1 commit into
apache:masterfrom
HyukjinKwon:minor-fix
Closed

[MINOR][DOCS] Use ASCII characters when possible in PySpark documentation#32048
HyukjinKwon wants to merge 1 commit into
apache:masterfrom
HyukjinKwon:minor-fix

Conversation

@HyukjinKwon

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR replaces the non-ASCII characters to ASCII characters when possible in PySpark documentation

Why are the changes needed?

To avoid unnecessarily using other non-ASCII characters which could lead to the issue such as #32047 or #22782

Does this PR introduce any user-facing change?

Virtually no.

How was this patch tested?

Found via (Mac OS):

# In Spark root directorycd python
pcregrep --color='auto' -n "[\x80-\xFF]"`git ls-files .`

@HyukjinKwon

Copy link
Copy Markdown
MemberAuthor

This doesn't have to be backported. So I intentionally separate it from #32047.

@SparkQA

Copy link
Copy Markdown

Test build #136890 has finished for PR 32048 at commit 8911f1d.

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

@SparkQA

Copy link
Copy Markdown

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

@SparkQA

Copy link
Copy Markdown

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

@MaxGekk

Copy link
Copy Markdown
Member

+1, LGTM. Merging to master.
Thank you @HyukjinKwon .

@HyukjinKwon

Copy link
Copy Markdown
MemberAuthor

Thx max!

@HyukjinKwon
HyukjinKwon deleted the minor-fix branch January 4, 2022 00:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@HyukjinKwon@SparkQA@MaxGekk