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.

Sign up for GitHub

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

Jump to bottom

Add synchronous replication docs #4947

Closed
wants to merge 5 commits into from
Closed

Add synchronous replication docs #4947

wants to merge 5 commits into from

Conversation

CharLotteiu
Copy link
Contributor

What is changed, added or deleted? (Required)

Add a document for the synchronous replication. Content migrated from tikv/tikv#7740 and tikv/website#211.

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

  • master (the latest development version)
  • 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)?

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Have version specific changes
  • Might cause conflicts

@CharLotteiu CharLotteiu added status/PTAL This PR is ready for reviewing. translation/from-docs This PR is translated from a PR in pingcap/docs. v5.0 This PR/issue applies to TiDB v5.0 labels Nov 23, 2020
@CharLotteiu CharLotteiu requested a review from TomShawn November 23, 2020 10:20
@ti-srebot
Copy link
Contributor

@TomShawn, PTAL.

@TomShawn TomShawn requested a review from BusyJay November 26, 2020 02:44
TomShawn
TomShawn reviewed Nov 26, 2020
View reviewed changes
Copy link
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

Minor comments on wording
@BusyJay PTAL

pd-configuration-file.md Outdated Show resolved Hide resolved
synchronous-replication.md Outdated Show resolved Hide resolved
synchronous-replication.md Show resolved Hide resolved
synchronous-replication.md Outdated Show resolved Hide resolved
synchronous-replication.md Outdated Show resolved Hide resolved
synchronous-replication.md Outdated Show resolved Hide resolved
CharLotteiu
CharLotteiu commented Nov 26, 2020
View reviewed changes
pd-control.md Outdated Show resolved Hide resolved
@TomShawn
Copy link
Contributor

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Nov 26, 2020
@ti-srebot
Copy link
Contributor

@TomShawn, @BusyJay, PTAL.

1 similar comment
@ti-srebot
Copy link
Contributor

@TomShawn, @BusyJay, PTAL.

@TomShawn
Copy link
Contributor

TomShawn commented Dec 1, 2020

After discussion with @BusyJay, we find that the feature name synchronous replication does not accurately describe this feature. @BusyJay is thinking of a new name and will update the English version first. So this PR is pending until the feature name is determined.

@TomShawn TomShawn added the status/WIP This PR is still working in progress. label Dec 1, 2020
@ti-srebot
Copy link
Contributor

@TomShawn, @BusyJay, PTAL.

@yikeke yikeke added for-future-release This PR only applies to master for now and might cherry-pick to a future release. and removed v5.0 This PR/issue applies to TiDB v5.0 labels Mar 24, 2021
@ti-chi-bot ti-chi-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. and removed status/WIP This PR is still working in progress. labels Apr 12, 2021
@TomShawn
Copy link
Contributor

TomShawn commented May 7, 2021

After discussion with @BusyJay, we find that the feature name synchronous replication does not accurately describe this feature. @BusyJay is thinking of a new name and will update the English version first. So this PR is pending until the feature name is determined.

@BusyJay Any update?

@BusyJay
Copy link
Contributor

BusyJay commented May 7, 2021

The feature will be release in following sprints, when the docs can be updated.

@qiancai
Copy link
Collaborator

qiancai commented May 31, 2021

Hi @BusyJay, we are preparing the user docs for TiDB v5.1 and would like to know whether the doc changes of this PR apply to TiDB v5.1 or not. Would you please check and reply? Thanks!

@ti-chi-bot ti-chi-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 23, 2021
@ti-chi-bot
Copy link
Member

@CharLotteiu: PR needs rebase.

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 kubernetes/test-infra repository.

@TomShawn
Copy link
Contributor

@BusyJay Any update?

@TomShawn TomShawn removed status/LGT1 Indicates that a PR has LGTM 1. translation/from-docs This PR is translated from a PR in pingcap/docs. status/PTAL This PR is ready for reviewing. needs-cherry-pick-release-5.0 for-future-release This PR only applies to master for now and might cherry-pick to a future release. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Aug 13, 2021
@ti-chi-bot ti-chi-bot added status/LGT1 Indicates that a PR has LGTM 1. missing-translation-status This PR does not have translation status info. labels Aug 13, 2021
@TomShawn
Copy link
Contributor

This PR is superseded by #6843.

@TomShawn TomShawn closed this Aug 13, 2021
@ti-chi-bot ti-chi-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 13, 2021
@ti-chi-bot
Copy link
Member

@CharLotteiu: PR needs rebase.

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 kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomShawn TomShawn TomShawn left review comments

@BusyJay BusyJay Awaiting requested review from BusyJay

Assignees
No one assigned
Labels
missing-translation-status This PR does not have translation status info. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@CharLotteiu @ti-srebot @TomShawn @BusyJay @qiancai @ti-chi-bot @yikeke