Skip to content

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/

cloud-fan and others added 2 commits February 24, 2021 18:18
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
@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 <cloud0fan@gmail.com>
Co-authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
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