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

zh,en: bring relay log back #518

Merged
merged 13 commits into from
Dec 25, 2020
Merged

Conversation

GMHDBJD
Copy link
Collaborator

@GMHDBJD GMHDBJD commented Dec 18, 2020

What is changed, added, or deleted? (Required)

bring relay log back

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

  • master (the latest development version, including v3.0 changes)
  • v2.0 (TiDB DM 2.0 versions)
  • v1.0 (TiDB DM 1.0 versions)

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

pingcap/dm#1291

  • 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

Copy link
Member

@csuzhangxc csuzhangxc left a comment

Choose a reason for hiding this comment

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

rest LGTM

Method 1: Use the command-line options.
```yaml
# relay log purge strategy
purge:
Copy link
Member

Choose a reason for hiding this comment

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

for YAML, you can update the following config items, like purge-interval to purge.interval.


```yaml
# relay log purge strategy
purge:
Copy link
Member

Choose a reason for hiding this comment

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

ditto

Copy link
Member

@csuzhangxc csuzhangxc left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Dec 18, 2020
@csuzhangxc csuzhangxc mentioned this pull request Dec 18, 2020
7 tasks
@CharLotteiu CharLotteiu added needs-cherry-pick-release-2.0 Should cherry pick this PR to release-2.0 branch. status/PTAL This PR is ready for reviewing. translation/no-need The changes in this PR don't need to be translated. labels Dec 18, 2020
en/relay-log.md Outdated Show resolved Hide resolved
zh/relay-log.md Outdated Show resolved Hide resolved
en/source-configuration-file.md Outdated Show resolved Hide resolved

> **Note:**
>
> The automatic cleanup policy will only take effect when `interval` is not 0 and at least one of the two configuration items `expires` and `remain-space` is not 0.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> The automatic cleanup policy will only take effect when `interval` is not 0 and at least one of the two configuration items `expires` and `remain-space` is not 0.
> The automatic data purge strategy only takes effect when `interval` is not 0 and at least one of the two configuration items `expires` and `remain-space` is not 0.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

address in ebb7bb3

zh/source-configuration-file.md Outdated Show resolved Hide resolved
@ti-srebot
Copy link
Contributor

@csuzhangxc, @CharLotteiu, @yikeke, PTAL.

GMHDBJD and others added 6 commits December 21, 2020 14:43
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
@ti-srebot
Copy link
Contributor

@yikeke, PTAL.

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

Rest LGTM

en/relay-log.md Outdated Show resolved Hide resolved
en/source-configuration-file.md Outdated Show resolved Hide resolved
GMHDBJD and others added 5 commits December 24, 2020 16:28
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
@lance6716
Copy link
Contributor

LGTM

@ti-srebot ti-srebot removed the status/LGT1 Indicates that a PR has LGTM 1. label Dec 25, 2020
@ti-srebot ti-srebot added the status/LGT2 Indicates that a PR has LGTM 2. label Dec 25, 2020
@CharLotteiu CharLotteiu merged commit 50ff512 into pingcap:master Dec 25, 2020
ti-srebot pushed a commit to ti-srebot/docs-dm that referenced this pull request Dec 25, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Dec 25, 2020
7 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-2.0 in PR #528

CharLotteiu pushed a commit that referenced this pull request Dec 25, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

Co-authored-by: GMHDBJD <35025882+GMHDBJD@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-cherry-pick-release-2.0 Should cherry pick this PR to release-2.0 branch. status/LGT2 Indicates that a PR has LGTM 2. status/PTAL This PR is ready for reviewing. translation/no-need The changes in this PR don't need to be translated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants