docs: update sync-diff-inspector docs for move to tiflow repo #20140
+24
−9
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Update GitHub repository links from tidb-tools to tiflow - Add TiUP installation method for v9.0.0+ - Update Docker image references - Add version-specific installation instructions - Update download-ecosystem-tools.md with new package information
ti-chi-bot
bot
added
missing-translation-status
ti-chi-bot
bot
added
size/M
dveeden
approved these changes
Jan 24, 2025
Oreoxmt
added
translation/doing
ti-chi-bot
bot
removed
the
missing-translation-status
Update Docker image for sync-diff-inspector The previous `docker pull pingcap/tiflow:latest` command was incorrect. After confirming with Flare Zuo (R&D Group - Engineering Productivity - Efficiency & Ops team), we should use `docker pull pingcap/sync-diff-inspector:latest` to align with TiCDC and DM, ensuring consistency across TiFlow-managed images. This update reflects the correct image name for better usability.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@dveeden Thanks for catching this! I initially assumed I’ve updated the docs accordingly in commit 8032c47. Let me know if you have any concerns! |
Oreoxmt
approved these changes
Feb 26, 2025
ti-chi-bot
bot
added
lgtm
and removed
needs-1-more-lgtm
15 tasks
Oreoxmt
added
area/sync-diff-inspector
Oreoxmt
added
the
type/compatibility-or-feature-change
|
/cc @qiancai |
qiancai
reviewed
Mar 7, 2025
| @@ -46,7 +46,7 @@ Depending on which tools you want to use, you can install the corresponding offl | |||
| | [TiDB Data Migration (DM)](/dm/dm-overview.md) | `dm-worker-{version}-linux-{arch}.tar.gz` <br/>`dm-master-{version}-linux-{arch}.tar.gz` <br/>`dmctl-{version}-linux-{arch}.tar.gz` | | |||
| | [TiCDC](/ticdc/ticdc-overview.md) | `cdc-{version}-linux-{arch}.tar.gz` | | |||
| | [Backup & Restore (BR)](/br/backup-and-restore-overview.md) | `br-{version}-linux-{arch}.tar.gz` | | |||
| | [sync-diff-inspector](/sync-diff-inspector/sync-diff-inspector-overview.md) | `sync_diff_inspector` | | |||
| | [sync-diff-inspector](/sync-diff-inspector/sync-diff-inspector-overview.md) | For TiDB v9.0.0 and later versions: included in TiUP and `tiflow-{version}-linux-{arch}.tar.gz` <br/>For TiDB versions before v9.0.0: `sync_diff_inspector` from tidb-tools | | |||
There was a problem hiding this comment.
This line looks confusing. It might cause the misunderstanding that other tools are not included in tiup
Suggested change
| | [sync-diff-inspector](/sync-diff-inspector/sync-diff-inspector-overview.md) | For TiDB v9.0.0 and later versions: included in TiUP and `tiflow-{version}-linux-{arch}.tar.gz` <br/>For TiDB versions before v9.0.0: `sync_diff_inspector` from tidb-tools | | |
| | [sync-diff-inspector](/sync-diff-inspector/sync-diff-inspector-overview.md) | For TiDB v9.0.0 and later versions: `tiflow-{version}-linux-{arch}.tar.gz` <br/>For TiDB versions before v9.0.0: `sync_diff_inspector` | |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is changed, added or deleted? (Required)
This PR updates the documentation to reflect the move of sync_diff_inspector from tidb-tools to tiflow repository for TiDB 9.0.0 and later versions. The changes include:
Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?