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

feat(tianmu): support 'ALTER TABLE t1 CHARACTER SET = ...' clause. (#848) #865

Merged
merged 2 commits into from
Nov 3, 2022

Conversation

lujiashun
Copy link

@lujiashun lujiashun commented Nov 1, 2022

Summary about this PR

Issue Number: close #848

1 add implement in check_if_supported_inplace_alter of tianmu handler;
2 add implement in inplace_alter_table of tianmu handler;
3 add implement in commit_inplace_alter_table of tianmu handler;
4. add mtr of ’atler table ... character set/atler table modify ... character set/atler table change ...character set';

https://dev.mysql.com/doc/refman/5.7/en/alter-table.html#alter-table-character-set

To change only the default character set for a table, use this statement:
ALTER TABLE tbl_name DEFAULT CHARACTER SET charset_name;
The word DEFAULT is optional. The default character set is the character set that is used if you do not
specify the character set for columns that you add to a table later (for example, with ALTER TABLE ...
ADD column)

Tests Check List

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Changelog

  • New Feature
  • Bug Fix
  • Improvement
  • Performance Improvement
  • Build/Testing/CI/CD
  • Documentation
  • Not for changelog (changelog entry is not required)

Documentation

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features

@mergify
Copy link
Contributor

mergify bot commented Nov 1, 2022

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@mergify mergify bot added the PR-feature feature for pull request label Nov 1, 2022
@RingsC
Copy link
Contributor

RingsC commented Nov 2, 2022

@lujiashun , Pls, add more specific information about this PR. such as this PR can slove what and not , etc. Thanks.

Copy link
Contributor

@RingsC RingsC left a comment

Choose a reason for hiding this comment

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

LGTM

@lujiashun lujiashun marked this pull request as draft November 2, 2022 08:11
@lujiashun lujiashun force-pushed the feat-848 branch 2 times, most recently from 232ced8 to 478cf5e Compare November 3, 2022 02:41
…toneatom#848)

[summary]
1 add implement in check_if_supported_inplace_alter of tianmu handler;
2 add implement in inplace_alter_table of tianmu handler;
3 add implement in commit_inplace_alter_table of tianmu handler;
@lujiashun lujiashun marked this pull request as ready for review November 3, 2022 05:23
Copy link
Collaborator

@isredstar isredstar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@hustjieke hustjieke left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit cf1c98c into stoneatom:stonedb-5.7-dev Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-feature feature for pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(tianmu): support 'ALTER TABLE t1 CHARACTER SET = ...' clause
4 participants