Skip to content

i18n(ja): repair sentences scrambled by the translation pipeline#23320

Draft
yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:i18n-ja-fix-scrambled-sentences
Draft

i18n(ja): repair sentences scrambled by the translation pipeline#23320
yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:i18n-ja-fix-scrambled-sentences

Conversation

@yahonda

@yahonda yahonda commented Jul 21, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Sites where the old MT pipeline scrambled the sentence itself — transposed or duplicated code spans, tore a list apart, or left its internal markers in the text. A missing particle can be inserted; these cannot, because the word order is wrong. Each sentence was rewritten against the English source.

25 lines across 18 files, in two groups.

1. Scrambled word order (11 lines)

EN: If you cannot confirm the condition that the query result falls in only one partition
    (such as `in`, `between`, `>`, `<`, `>=`, `<=`), you cannot use the partition pruning optimization.
JA: クエリ結果`between` 1つの`>` `in` )のみに該当するという条件を確認できない場合は、
    パーティションプルーニング最適化`<=`使用できません。 `<` `>=`
->  クエリ結果が1つのパーティションのみに該当するという条件( `in` 、 `between` 、 `>` 、 `<` 、 `>=` 、 `<=`など)を
    確認できない場合は、パーティションプルーニング最適化を使用できません。例:

Three of the six operators had been stranded after the sentence-final period; the word パーティション and the trailing 例: had been swallowed.

Others in this group carried wrong information rather than just bad grammar:

file was now
functions-and-operators/window-functions.md 7がnの場合 nが1の場合 — EN "When n is 1"
tiflash/troubleshoot-tiflash.md …を返して次のステップに進みます …を実行してから次のステップに進みます — EN's "and" had become 返す
ticdc/ticdc-open-api-v2.md 詳細は3のセクションを参照してください 詳細はそのセクションを参照してください — EN "See that section"
sql-statements/sql-statement-drop-index.md `CLUSTERED`型`CLUSTERED`主キー `CLUSTERED`型の主キー — duplicated span

2. Leftover tool markers (14 lines)

{{B, {{B-, -E}} — the same pipeline artifact as the PLACEHOLDER strings fixed in #23315, but in a different surface form that a grep PLACEHOLDER misses. The English source contains zero occurrences of either, so all of it is garbage; in every case it had swallowed words and shuffled the neighbouring spans.

JA: TiDB は`SELECT` 、 `USE` }、{{B- `SHOW` -E}} など、データを変更しないステートメントのみを実行します。
->  TiDB は`SELECT` 、 `USE` 、 `SHOW` など、データを変更しないステートメントのみを実行します。

Legitimate }} in JSON and log samples was excluded from the sweep and left untouched.

How it was checked

  • Every site read against release-8.5 before editing.
  • Per changed line, the multiset of code spans and of link URLs is unchanged — spans were only repositioned (or, for a duplicate, the extra one removed). 0 invented spans, 0 altered URLs.
  • No doubled particle introduced. grep -rn '{{B\|PLACEHOLDER\|-E}}': 0 remaining across the whole tree.

Relationship to the sibling PRs

Developed on top of the other open ja PRs and harmonized with them: all fourteen merge cleanly in any order (verified — 91 pairwise merges, 0 conflicts).

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

  • master (the latest development version)
  • v8.5 (TiDB 8.5 versions)
  • v8.4 (TiDB 8.4 versions)
  • v8.3 (TiDB 8.3 versions)
  • v8.2 (TiDB 8.2 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 versions)
  • v5.3 (TiDB 5.3 versions)

What is the related PR or file 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

ti-chi-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ti-chi-bot ti-chi-bot Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9b0e4ffb-0961-4709-acab-71ec12e24f5d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ti-chi-bot ti-chi-bot Bot added missing-translation-status This PR does not have translation status info. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 21, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign icemap for approval. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

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

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sites where the old MT pipeline transposed or duplicated code spans, tore a list
apart, or left its internal markers ({{B, -E}}) in the text. These cannot be fixed
by inserting a particle — the word order itself is wrong — so each sentence was
rewritten against the English source.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@yahonda
yahonda force-pushed the i18n-ja-fix-scrambled-sentences branch from 25d76d2 to 58ed761 Compare July 21, 2026 07:18
@ti-chi-bot ti-chi-bot Bot added area/develop This PR relates to the area of TiDB App development. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 21, 2026
@qiancai qiancai added translation/no-need No need to translate this PR. and removed missing-translation-status This PR does not have translation status info. labels Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/develop This PR relates to the area of TiDB App development. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants