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

[doc](auto-inc) Add user oriented doc for auto increment column #29230

Merged
merged 17 commits into from
Jan 12, 2024

Conversation

bobhan1
Copy link
Contributor

@bobhan1 bobhan1 commented Dec 28, 2023

Proposed changes

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

@bobhan1
Copy link
Contributor Author

bobhan1 commented Dec 28, 2023

run buildall


1. 仅Duplicate模型表和Unique模型表可以包含自增列。
2. 一张表最多只能包含一个自增列。
3. 自增列的类型必须是BIGINT类型,且必须为NOT NULL。
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it support schema change?

@bobhan1 bobhan1 requested a review from zhannngchen December 28, 2023 11:35
@bobhan1
Copy link
Contributor Author

bobhan1 commented Dec 29, 2023

run buildall

@bobhan1 bobhan1 requested a review from dataroaring December 29, 2023 08:37
@bobhan1
Copy link
Contributor Author

bobhan1 commented Dec 29, 2023

run buildall

@bobhan1 bobhan1 force-pushed the auto-inc-doc branch 5 times, most recently from f47679b to ccb2141 Compare December 29, 2023 13:59
dataroaring
dataroaring previously approved these changes Jan 11, 2024
Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@dataroaring
Copy link
Contributor

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jan 11, 2024
Copy link
Contributor

PR approved by at least one committer and no changes requested.

Copy link
Contributor

PR approved by anyone and no changes requested.

Copy link
Contributor

@zhannngchen zhannngchen left a comment

Choose a reason for hiding this comment

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

LGTM

@dataroaring
Copy link
Contributor

run buildall

zhannngchen
zhannngchen previously approved these changes Jan 12, 2024
@bobhan1
Copy link
Contributor Author

bobhan1 commented Jan 12, 2024

run buildall

Copy link
Contributor

@zhannngchen zhannngchen 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
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@zhannngchen zhannngchen merged commit 5f86b7b into apache:master Jan 12, 2024
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. dev/3.0.0-merged reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants