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

[SPARK-31891][SQL][DOCS][FOLLOWUP] Fix typo in the description of MSCK REPAIR TABLE #31633

Closed

Conversation

MaxGekk
Copy link
Member

@MaxGekk MaxGekk commented Feb 24, 2021

What changes were proposed in this pull request?

Fix typo and highlight that ADD PARTITIONS is the default.

Why are the changes needed?

Fix a typo which can mislead users.

Does this PR introduce any user-facing change?

No

How was this patch tested?

n/a

@MaxGekk
Copy link
Member Author

MaxGekk commented Feb 24, 2021

@cloud-fan @dongjoon-hyun Please, review this PR.

@github-actions github-actions bot added the DOCS label Feb 24, 2021
@SparkQA
Copy link

SparkQA commented Feb 24, 2021

Test build #135411 has finished for PR 31633 at commit 8441439.

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

@SparkQA
Copy link

SparkQA commented Feb 24, 2021

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

@SparkQA
Copy link

SparkQA commented Feb 24, 2021

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

@SparkQA
Copy link

SparkQA commented Feb 24, 2021

Test build #135418 has finished for PR 31633 at commit 0101c62.

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

@SparkQA
Copy link

SparkQA commented Feb 24, 2021

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

@SparkQA
Copy link

SparkQA commented Feb 24, 2021

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

xuanyuanking pushed a commit to xuanyuanking/spark that referenced this pull request Sep 29, 2021
…CK REPAIR TABLE`

### What changes were proposed in this pull request?
Fix typo and highlight that `ADD PARTITIONS` is the default.

### Why are the changes needed?
Fix a typo which can mislead users.

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

### How was this patch tested?
n/a

Closes apache#31633 from MaxGekk/repair-table-drop-partitions-followup.

Lead-authored-by: Wenchen Fan <[email protected]>
Co-authored-by: Max Gekk <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
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.

4 participants