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

fix: remove warning for "REORGANIZE PARTITION" (#605) #924

Merged
merged 2 commits into from
Jul 18, 2020

Conversation

wangggong
Copy link
Contributor

Note that in commit fdfeac in pingcap/tidb the errror was already handled.
Need to do nothing.

Signed-off-by: Wang Ruichao [email protected]

What problem does this PR solve?

Close #605 (cc #598)

What is changed and how it works?

Update judgement in parser.y.

Note that in my opinion "REORGANIZE PARTITION" is a part of partition options, I moved it to "AlterTablePartitionOpt". If not, I'll rollback.

Check List

Tests

  • Unit test

Note that in commit fdfeac in pingcap/tidb the errror was already handled.
Need to do nothing.

Signed-off-by: Wang Ruichao <[email protected]>
@codecov
Copy link

codecov bot commented Jul 11, 2020

Codecov Report

Merging #924 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #924      +/-   ##
==========================================
- Coverage   78.43%   78.42%   -0.01%     
==========================================
  Files          40       40              
  Lines       14897    14775     -122     
==========================================
- Hits        11685    11588      -97     
+ Misses       2521     2500      -21     
+ Partials      691      687       -4     

Copy link
Contributor

@kennytm kennytm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kennytm kennytm added the status/LGT1 LGT1 label Jul 14, 2020
Copy link
Contributor

@djshow832 djshow832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original issue says an error should be reported. But this PR just removes the warning. I don't think it's resolving the issue.

@tangenta
Copy link
Contributor

Copy link
Contributor

@tangenta tangenta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kennytm kennytm added status/LGT2 LGT2 and removed status/LGT1 LGT1 labels Jul 18, 2020
@kennytm kennytm merged commit 3dd0dd2 into pingcap:master Jul 18, 2020
@wangggong wangggong deleted the recorganize_partition branch July 18, 2020 14:04
tiancaiamao pushed a commit to tiancaiamao/parser that referenced this pull request Apr 27, 2021
…#924)

Note that in commit fdfeac in pingcap/tidb the errror was already handled.
Need to do nothing.

Signed-off-by: Wang Ruichao <[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.

Fix TiDB behavior: reorganize partition
5 participants