Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MINOR][SQL] Fix the typo in function names: crete #34978

Closed
wants to merge 1 commit into from

Conversation

MaxGekk
Copy link
Member

@MaxGekk MaxGekk commented Dec 21, 2021

What changes were proposed in this pull request?

Fix the typo: crete -> create.

Why are the changes needed?

To improve code maintenance. Find the functions by names should be easer after the changes.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

By compiling and running related test suites:

$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"

@SparkQA
Copy link

SparkQA commented Dec 21, 2021

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

@SparkQA
Copy link

SparkQA commented Dec 21, 2021

Test build #146450 has finished for PR 34978 at commit eded731.

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

@SparkQA
Copy link

SparkQA commented Dec 21, 2021

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

HyukjinKwon pushed a commit that referenced this pull request Dec 22, 2021
### What changes were proposed in this pull request?
Fix the typo: crete -> create.

### Why are the changes needed?
To improve code maintenance. Find the functions by names should be easer after the changes.

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

### How was this patch tested?
By compiling and running related test suites:
```
$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"
```

Closes #34978 from MaxGekk/fix-typo-crete.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 72c278a)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
HyukjinKwon pushed a commit that referenced this pull request Dec 22, 2021
### What changes were proposed in this pull request?
Fix the typo: crete -> create.

### Why are the changes needed?
To improve code maintenance. Find the functions by names should be easer after the changes.

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

### How was this patch tested?
By compiling and running related test suites:
```
$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"
```

Closes #34978 from MaxGekk/fix-typo-crete.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 72c278a)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@HyukjinKwon
Copy link
Member

Merged to master, branch-3.2 and branch-3.1.

catalinii pushed a commit to lyft/spark that referenced this pull request Feb 22, 2022
### What changes were proposed in this pull request?
Fix the typo: crete -> create.

### Why are the changes needed?
To improve code maintenance. Find the functions by names should be easer after the changes.

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

### How was this patch tested?
By compiling and running related test suites:
```
$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"
```

Closes apache#34978 from MaxGekk/fix-typo-crete.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 72c278a)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
catalinii pushed a commit to lyft/spark that referenced this pull request Mar 4, 2022
### What changes were proposed in this pull request?
Fix the typo: crete -> create.

### Why are the changes needed?
To improve code maintenance. Find the functions by names should be easer after the changes.

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

### How was this patch tested?
By compiling and running related test suites:
```
$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"
```

Closes apache#34978 from MaxGekk/fix-typo-crete.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 72c278a)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
kazuyukitanimura pushed a commit to kazuyukitanimura/spark that referenced this pull request Aug 10, 2022
### What changes were proposed in this pull request?
Fix the typo: crete -> create.

### Why are the changes needed?
To improve code maintenance. Find the functions by names should be easer after the changes.

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

### How was this patch tested?
By compiling and running related test suites:
```
$ build/sbt "test:testOnly *ParquetRebaseDatetimeV2Suite"
$ build/sbt "test:testOnly *AvroV1Suite"
```

Closes apache#34978 from MaxGekk/fix-typo-crete.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 72c278a)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants