Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(sql-check):support part of modify data type online ddl check #4170

Open
wants to merge 2 commits into
base: dev/4.3.x
Choose a base branch
from

Conversation

LioRoger
Copy link
Collaborator

What type of PR is this?

type-feature

What this PR does / why we need it:

if OBMySQL version greater than 4.0.0, check if data type change statement is online ddl

  1. modify (add/remove/change)default value
  2. change null flag null / not null
  3. increase precision of char/varchar/text/number
    collate/charset modify is offline ddl for now.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


@LioRoger LioRoger added the type-feature The functionality to be implemented label Jan 15, 2025
@LioRoger LioRoger added this to the ODC 4.3.4 milestone Jan 15, 2025
Copy link
Collaborator

@guowl3 guowl3 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

@zijiacj zijiacj left a comment

Choose a reason for hiding this comment

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

lgtm

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type-feature The functionality to be implemented
Projects
Status: New
Development

Successfully merging this pull request may close these issues.

4 participants