Skip to content

faq: add info for schema is changed error#7043

Merged
ti-chi-bot merged 4 commits into
pingcap:masterfrom
zimulala:zimuxia/schema-changed
Sep 10, 2021
Merged

faq: add info for schema is changed error#7043
ti-chi-bot merged 4 commits into
pingcap:masterfrom
zimulala:zimuxia/schema-changed

Conversation

@zimulala
Copy link
Copy Markdown
Contributor

@zimulala zimulala commented Sep 1, 2021 •
edited by TomShawn
Loading

First-time contributors' checklist

What is changed, added or deleted? (Required)

Which TiDB version(s) do your changes apply to? (Required)

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions (in Chinese).

  • master (the latest development version)
  • v5.2 (TiDB 5.2 versions)
  • v5.1 (TiDB 5.1 versions)
  • v5.0 (TiDB 5.0 versions)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 1, 2021 •
edited
Loading

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • TomShawn

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Details

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot requested a review from TomShawn September 1, 2021 02:27
@ti-chi-bot ti-chi-bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 1, 2021
TomShawn
TomShawn reviewed Sep 1, 2021
View reviewed changes
Comment thread faq/sql-faq.md Outdated
- 执行的 DML 语句中涉及的表和集群中正在执行的 DDL 的表有相同的，那么这个 DML 语句就会报此错。
- 这个 DML 执行时间很久，而这段时间内执行了很多 DDL 语句，导致中间 `schema` 版本变更次数超过 1024 （此为默认值，可以通过 `tidb_max_delta_schema_count` 变量修改）。
- 接受 DML 请求的 TiDB 长时间不能加载到 `schema information`（TiDB 与 PD 或 TiKV 之间的网络连接故障等会导致此问题），而这段时间内执行了很多 DDL 语句，导致中间 `schema` 版本变更次数超过 100。
- TiDB 重启后遇到第一个 DDL 操作期间，执行 DML 并且在它执行过程中遇到了 DDL 操作，那么这个 DML 很可能会报此错。
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn Sep 1, 2021

Choose a reason for hiding this comment

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

“第一个 DDL 操作”和“执行过程中遇到了 DDL 操作”，这两个是同一个 DDL 操作吗？

Copy link
Copy Markdown
Contributor Author

@zimulala zimulala Sep 1, 2021

Choose a reason for hiding this comment

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

可能遇到多个 DDL，第一个 DDL 一定会包含。我改一下描述

Copy link
Copy Markdown
Contributor

@TomShawn TomShawn Sep 1, 2021

Choose a reason for hiding this comment

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

能否请一位研发老师看看呢？谢谢~

Copy link
Copy Markdown
Contributor Author

@zimulala zimulala Sep 1, 2021

Choose a reason for hiding this comment

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

PTAL @tiancaiamao @qw4990

@TomShawn TomShawn removed needs-cherry-pick-release-3.1 missing-translation-status This PR does not have translation status info. labels Sep 1, 2021
@TomShawn TomShawn added sig/docs Indicates that the Issue or PR belongs to the docs SIG. area/sql-infra Indicates that the Issue or PR belongs to the area of sql-infra and sql-metadata. translation/doing This PR’s assignee is translating this PR. labels Sep 1, 2021
@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 1, 2021

@qw4990: Thanks for your review. The bot only counts approvals from reviewers and higher roles in list, but you're still welcome to leave your comments.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

TomShawn
TomShawn reviewed Sep 6, 2021
View reviewed changes
Comment thread faq/sql-faq.md Outdated
@zimulala @TomShawn
Update faq/sql-faq.md
49f6ef2 
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@zimulala
Copy link
Copy Markdown
Contributor Author

zimulala commented Sep 10, 2021

PTAL @TomShawn

@ti-chi-bot ti-chi-bot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 10, 2021
@TomShawn
Copy link
Copy Markdown
Contributor

TomShawn commented Sep 10, 2021

/merge

@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 10, 2021

This pull request has been accepted and is ready to merge.

DetailsCommit hash: 49f6ef2

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Sep 10, 2021
@ti-chi-bot ti-chi-bot merged commit bab9cd3 into pingcap:master Sep 10, 2021
ti-chi-bot pushed a commit to ti-chi-bot/docs-cn that referenced this pull request Sep 10, 2021
@zimulala @ti-chi-bot
faq: add info for schema is changed error (pingcap#7043)
7965c5b
@ti-chi-bot ti-chi-bot mentioned this pull request Sep 10, 2021
Merged
13 tasks
@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 10, 2021

In response to a cherrypick label: new pull request created: #7100.

ti-chi-bot pushed a commit to ti-chi-bot/docs-cn that referenced this pull request Sep 10, 2021
@zimulala @ti-chi-bot
faq: add info for schema is changed error (pingcap#7043)
161ef12
@ti-chi-bot ti-chi-bot mentioned this pull request Sep 10, 2021
Merged
13 tasks
@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 10, 2021

In response to a cherrypick label: new pull request created: #7101.

ti-chi-bot pushed a commit to ti-chi-bot/docs-cn that referenced this pull request Sep 10, 2021
@zimulala @ti-chi-bot
faq: add info for schema is changed error (pingcap#7043)
35e1371
@ti-chi-bot ti-chi-bot mentioned this pull request Sep 10, 2021
Merged
13 tasks
@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 10, 2021

In response to a cherrypick label: new pull request created: #7102.

ti-chi-bot pushed a commit to ti-chi-bot/docs-cn that referenced this pull request Sep 10, 2021
@zimulala @ti-chi-bot
faq: add info for schema is changed error (pingcap#7043)
3d6bb0b
@ti-chi-bot ti-chi-bot mentioned this pull request Sep 10, 2021
Merged
13 tasks
@ti-chi-bot
Copy link
Copy Markdown
Member

ti-chi-bot commented Sep 10, 2021

In response to a cherrypick label: new pull request created: #7103.

ti-chi-bot added a commit that referenced this pull request Sep 10, 2021
@ti-chi-bot
faq: add info for schema is changed error (#7043) (#7101)
7dbb9fd
ti-chi-bot added a commit that referenced this pull request Sep 10, 2021
@ti-chi-bot
faq: add info for schema is changed error (#7043) (#7102)
652646f
ti-chi-bot added a commit that referenced this pull request Sep 10, 2021
@ti-chi-bot
faq: add info for schema is changed error (#7043) (#7103)
2d1bb15
ti-chi-bot added a commit that referenced this pull request Sep 10, 2021
@ti-chi-bot
faq: add info for schema is changed error (#7043) (#7100)
80e9364
@Liuxiaozhen12 Liuxiaozhen12 added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Sep 13, 2021
@Liuxiaozhen12 Liuxiaozhen12 mentioned this pull request Sep 14, 2021
Merged
13 tasks
This was referenced Sep 15, 2021
Merged
Merged
Merged
Merged
@zimulala zimulala deleted the zimuxia/schema-changed branch June 19, 2023 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qw4990 qw4990 qw4990 approved these changes

+1 more reviewer

@TomShawn TomShawn TomShawn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Liuxiaozhen12 Liuxiaozhen12

Labels

area/sql-infra Indicates that the Issue or PR belongs to the area of sql-infra and sql-metadata. sig/docs Indicates that the Issue or PR belongs to the docs SIG. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@zimulala @ti-chi-bot @TomShawn @qw4990 @Liuxiaozhen12