Skip to content

[MINOR] Spelling sql/core#30531

Closed
jsoref wants to merge 63 commits into
apache:masterfrom
jsoref:spelling-sql-core
Closed

[MINOR] Spelling sql/core#30531
jsoref wants to merge 63 commits into
apache:masterfrom
jsoref:spelling-sql-core

Conversation

@jsoref

@jsoref jsoref commented Nov 28, 2020

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR intends to fix typos in the sub-modules:

  • sql/core

Split per @srowen #30323 (comment)

NOTE: The misspellings have been reported at jsoref@706a726#commitcomment-44064356

Why are the changes needed?

Misspelled words make it harder to read / understand content.

Does this PR introduce any user-facing change?

There are various fixes to documentation, etc...

How was this patch tested?

No testing was performed

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.

I don't feel strongly either way, but these don't necessarily need to be renamed. They're intentionally shorthand it seems. It might just reduce the number of lines touched.

@srowen srowen changed the title Spelling sql/core [MINOR] Spelling sql/core Nov 29, 2020
@srowen

srowen commented Dec 2, 2020

Copy link
Copy Markdown
Member

I think this just needs a rebase, whenever prerequisite changes are in

jsoref added 21 commits December 7, 2020 10:06
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
jsoref added 16 commits December 7, 2020 10:08
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
@jsoref jsoref force-pushed the spelling-sql-core branch from e42315a to db599a5 Compare December 7, 2020 15:08

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

Looks OK if tests pass

@srowen

srowen commented Dec 7, 2020

Copy link
Copy Markdown
Member

Jenkins retest this please

@SparkQA

SparkQA commented Dec 7, 2020

Copy link
Copy Markdown

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

@SparkQA

SparkQA commented Dec 7, 2020

Copy link
Copy Markdown

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

@SparkQA

SparkQA commented Dec 7, 2020

Copy link
Copy Markdown

Test build #132379 has finished for PR 30531 at commit db599a5.

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

@srowen

srowen commented Dec 8, 2020

Copy link
Copy Markdown
Member

Merged to master. @jsoref are there other PRs besides the main one left now?

@srowen srowen closed this in a093d6f Dec 8, 2020
@jsoref jsoref deleted the spelling-sql-core branch December 8, 2020 15:18
@jsoref

jsoref commented Dec 8, 2020

Copy link
Copy Markdown
Contributor Author

Nope. I'll refresh and split that one today.

@dongjoon-hyun

Copy link
Copy Markdown
Member

Hi, @srowen and @jsoref . I'll also merge this minor patch to branch-3.1 to reduce merge conflicts.

cc @HyukjinKwon

@dongjoon-hyun

Copy link
Copy Markdown
Member

I locally finished compilation tests and pushed.

dongjoon-hyun pushed a commit that referenced this pull request Dec 22, 2020
### What changes were proposed in this pull request?

This PR intends to fix typos in the sub-modules:
* `sql/core`

Split per srowen #30323 (comment)

NOTE: The misspellings have been reported at jsoref@706a726#commitcomment-44064356

### Why are the changes needed?

Misspelled words make it harder to read / understand content.

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

There are various fixes to documentation, etc...

### How was this patch tested?

No testing was performed

Closes #30531 from jsoref/spelling-sql-core.

Authored-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Sean Owen <srowen@gmail.com>
(cherry picked from commit a093d6f)
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Sign up for free to 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.

4 participants