Skip to content

i18n(ja): fix dropped particles before verbs across the corpus (batch 2) - #23611

Merged
qiancai merged 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-dropped-particles-batch2-2026-08-26
Aug 28, 2026
Merged

i18n(ja): fix dropped particles before verbs across the corpus (batch 2)#23611
qiancai merged 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-dropped-particles-batch2-2026-08-26

Conversation

@yahonda

@yahonda yahonda commented Aug 26, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Corpus-wide sweep for sentences with a dropped particle (mainly を) immediately before a verb, following on from #23579's original pattern. Scope was expanded beyond the original ~14-verb heuristic to ~50 common verbs, and re-scanned across the whole i18n-ja-release-8.5 tree.

  • 190 files, one particle restored per changed line, verified against the English source for each site.
  • Excludes cases where a code-span/bracket immediately before the verb is in apposition to a noun already marked earlier in the same clause (would create a double-を) — these were checked individually and confirmed as false positives, not touched.

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

  • i18n-ja-release-8.5 (TiDB Japanese documentation for TiDB 8.5 versions)

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

AI agent involvement

  • The changes in this PR were primarily made by an AI agent on behalf of the PR author.

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

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Too many files!

This PR contains 190 files, which is 40 over the limit of 150.

To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch.

Upgrade to a paid plan to raise the limit.

This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 0a67df6e-6b3b-4694-aa8e-82a96bbafef9

📥 Commits

Reviewing files that changed from the base of the PR and between d41688a and 4f9089b.

📒 Files selected for processing (190)
  • ai/guides/vector-search-hybrid-search.md
  • ai/quickstart-via-python.md
  • auto-increment.md
  • auto-random.md
  • benchmark/benchmark-tidb-using-sysbench.md
  • best-practices/best-practices-on-public-cloud.md
  • best-practices/haproxy-best-practices.md
  • best-practices/massive-regions-best-practices.md
  • best-practices/pd-scheduling-best-practices.md
  • best-practices/tidb-partitioned-tables-best-practices.md
  • br/br-monitoring-and-alert.md
  • br/br-pitr-guide.md
  • br/br-pitr-manual.md
  • br/br-snapshot-guide.md
  • br/use-br-command-line-tool.md
  • certificate-authentication.md
  • clinic/quick-start-with-clinic.md
  • command-line-flags-for-scheduling-configuration.md
  • command-line-flags-for-tso-configuration.md
  • constraints.md
  • coprocessor-cache.md
  • daily-check.md
  • dashboard/top-sql.md
  • data-type-date-and-time.md
  • deploy-monitoring-services.md
  • develop/dev-guide-aws-appflow-integration.md
  • develop/dev-guide-connection-parameters.md
  • develop/dev-guide-optimistic-and-pessimistic-transaction.md
  • develop/dev-guide-paginate-results.md
  • develop/dev-guide-timeouts-in-tidb.md
  • develop/dev-guide-unstable-result-set.md
  • develop/dev-guide-use-stale-read.md
  • develop/java-app-best-practices.md
  • dm/dm-compatibility-catalog.md
  • dm/dm-error-handling.md
  • dm/dm-webui-guide.md
  • dm/feature-shard-merge-pessimistic.md
  • dm/relay-log.md
  • dr-multi-replica.md
  • enable-disk-spill-encrypt.md
  • enable-tls-between-components.md
  • encryption-at-rest.md
  • error-codes.md
  • explain-overview.md
  • faq/backup-and-restore-faq.md
  • faq/deploy-and-maintain-faq.md
  • faq/high-reliability-faq.md
  • faq/migration-tidb-faq.md
  • faq/sql-faq.md
  • faq/upgrade-faq.md
  • foreign-key.md
  • functions-and-operators/precision-math.md
  • functions-and-operators/set-operators.md
  • global-indexes.md
  • grafana-tidb-dashboard.md
  • log-redaction.md
  • migrate-with-pt-ghost.md
  • optimistic-transaction.md
  • performance-tuning-methods.md
  • performance-tuning-practices.md
  • releases/release-2.0-ga.md
  • releases/release-2.0-rc.4.md
  • releases/release-2.1-beta.md
  • releases/release-2.1-ga.md
  • releases/release-2.1-rc.1.md
  • releases/release-2.1-rc.2.md
  • releases/release-2.1.11.md
  • releases/release-3.0-ga.md
  • releases/release-3.0.16.md
  • releases/release-3.0.19.md
  • releases/release-3.0.2.md
  • releases/release-3.0.9.md
  • releases/release-4.0-ga.md
  • releases/release-4.0.0-rc.md
  • releases/release-4.0.11.md
  • releases/release-4.0.13.md
  • releases/release-4.0.3.md
  • releases/release-4.0.6.md
  • releases/release-4.0.8.md
  • releases/release-4.0.9.md
  • releases/release-5.0.0-rc.md
  • releases/release-5.0.2.md
  • releases/release-5.3.0.md
  • releases/release-6.1.0.md
  • releases/release-6.1.7.md
  • releases/release-6.3.0.md
  • releases/release-6.5.1.md
  • releases/release-6.5.4.md
  • releases/release-6.5.6.md
  • releases/release-7.1.0.md
  • releases/release-7.1.2.md
  • releases/release-7.1.3.md
  • releases/release-7.4.0.md
  • releases/release-7.5.1.md
  • releases/release-7.5.7.md
  • releases/release-7.6.0.md
  • releases/release-8.0.0.md
  • releases/release-8.1.0.md
  • releases/release-8.1.1.md
  • releases/release-8.1.2.md
  • releases/release-8.4.0.md
  • resources/markdownlint-rules.md
  • role-based-access-control.md
  • runtime-filter.md
  • schedule-replicas-by-topology-labels.md
  • scheduling-configuration-file.md
  • shard-row-id-bits.md
  • sql-plan-replayer.md
  • sql-prepared-plan-cache.md
  • sql-statements/sql-statement-add-column.md
  • sql-statements/sql-statement-admin-checksum-table.md
  • sql-statements/sql-statement-alter-index.md
  • sql-statements/sql-statement-alter-table.md
  • sql-statements/sql-statement-commit.md
  • sql-statements/sql-statement-create-index.md
  • sql-statements/sql-statement-flashback-database.md
  • sql-statements/sql-statement-load-data.md
  • sql-statements/sql-statement-lock-stats.md
  • sql-statements/sql-statement-lock-tables-and-unlock-tables.md
  • sql-statements/sql-statement-recover-table.md
  • sql-statements/sql-statement-set-role.md
  • stale-read.md
  • storage-engine/titan-configuration.md
  • storage-engine/titan-overview.md
  • sync-diff-inspector/sync-diff-inspector-overview.md
  • table-attributes.md
  • ticdc/ticdc-architecture.md
  • ticdc/ticdc-avro-protocol.md
  • ticdc/ticdc-canal-json.md
  • ticdc/ticdc-changefeed-overview.md
  • ticdc/ticdc-faq.md
  • ticdc/ticdc-filter.md
  • ticdc/ticdc-integrity-check.md
  • ticdc/ticdc-server-config.md
  • ticdc/ticdc-simple-protocol.md
  • ticdc/ticdc-sink-to-cloud-storage.md
  • ticdc/ticdc-sink-to-kafka.md
  • ticdc/ticdc-split-update-behavior.md
  • ticdc/ticdc-upstream-downstream-check.md
  • tidb-cloud/changefeed-sink-to-apache-pulsar.md
  • tidb-cloud/connected-lark-ticket-creation.md
  • tidb-cloud/csv-config-for-import-data.md
  • tidb-cloud/data-service-manage-endpoint.md
  • tidb-cloud/data-service-manage-github-connection.md
  • tidb-cloud/integrate-tidbcloud-with-zapier.md
  • tidb-cloud/limited-sql-features.md
  • tidb-cloud/manage-user-access.md
  • tidb-cloud/monitor-new-relic-integration.md
  • tidb-cloud/premium/built-in-monitoring-premium.md
  • tidb-cloud/premium/import-with-mysql-cli-premium.md
  • tidb-cloud/releases/release-notes-2022.md
  • tidb-cloud/releases/release-notes-2025.md
  • tidb-cloud/serverless-faqs.md
  • tidb-cloud/serverless-limitations.md
  • tidb-cloud/set-up-vpc-peering-connections.md
  • tidb-cloud/terraform-use-serverless-branch-resource.md
  • tidb-cloud/tidb-cloud-encrypt-cmek-aws.md
  • tidb-cloud/use-chat2query-api.md
  • tidb-configuration-file.md
  • tidb-distributed-execution-framework.md
  • tidb-external-ts.md
  • tidb-lightning/tidb-lightning-configuration.md
  • tidb-lightning/tidb-lightning-logical-import-mode.md
  • tidb-lightning/troubleshoot-tidb-lightning.md
  • tidb-read-staleness.md
  • tidb-resource-control-runaway-queries.md
  • tidb-scheduling.md
  • tidb-troubleshooting-map.md
  • tiflash/tiflash-configuration.md
  • tiflash/tiflash-spill-disk.md
  • tiflash/troubleshoot-tiflash.md
  • tiflash/use-fastscan.md
  • tiflash/use-tiflash-mpp-mode.md
  • tikv-control.md
  • tikv-in-memory-engine.md
  • tiproxy/tiproxy-command-line-flags.md
  • tiproxy/tiproxy-configuration.md
  • tiproxy/tiproxy-load-balance.md
  • tiproxy/tiproxy-performance-test.md
  • tiup/tiup-cluster-topology-reference.md
  • tiup/tiup-command-completion.md
  • tiup/tiup-component-management.md
  • transaction-isolation-levels.md
  • troubleshoot-cpu-issues.md
  • troubleshoot-data-inconsistency-errors.md
  • troubleshoot-high-disk-io.md
  • troubleshoot-tidb-cluster.md
  • troubleshoot-tidb-oom.md
  • tso-configuration-file.md
  • tune-region-performance.md

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


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. area/develop This PR relates to the area of TiDB App development. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 26, 2026
@ti-chi-bot

ti-chi-bot Bot commented Aug 26, 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

yahonda added a commit that referenced this pull request Aug 27, 2026
55 sites across 51 files where the topic particle は appears to be
missing immediately after a code span/bracket, before a definitional
or state predicate (becomes/is called/represents/corresponds to/is
supported/etc.).

NOT individually verified against the English source per-site, unlike
the previous を (#23579/#23611) and が (#23624) sweeps. は is far more
context-dependent than を/が (Japanese frequently and correctly omits
は when a topic is already established), so a mechanical detector has
a much higher false-positive rate here. Submitted as a draft
specifically so a human can review each site before merging.

UPDATE: user review caught a systemic false-positive class in the
になります/となります predicate category (18 sites) -- the captured
code span/bracket is the COMPLEMENT (the resulting value) of the
"becomes X" predicate, not its subject, e.g. "この機能が一般提供
(GA)となります" (the feature becomes GA) mis-edited to insert は
right before となります, producing the ungrammatical "一般提供(GA)
はとなります". All 18 sites in this predicate category were reverted;
37 sites across the other predicate categories remain.
Fixed missing particles (mostly を, a few が/は/で/に) immediately before
common action verbs (設定する, 有効にする, 追加する, 実行する, 使用する,
インストールする, 検索する, etc.) where a code-span, closing paren, or
bracket directly touches the verb with no particle in between. Combines
four parallel groups covering 190 files total.

- group 0: 47 particle insertions/corrections across 42 files
- group 1: ~55 particle insertions across 49 files
- group 2: 71 particle insertions/fixes across 46 files
- group 3: ~48 particle insertions across 51 files

A number of adjacent pre-existing MT defects were also fixed alongside
the flagged particles when directly confirmed against the release-8.5
EN source, including scrambled/reordered code-span lists, duplicated or
wrong verbs, garbled/duplicated link text, and misplaced PR-link/terms
breaking sentences. See the individual group commits on the source
branch for the full per-file breakdown.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@yahonda
yahonda force-pushed the fix-dropped-particles-batch2-2026-08-26 branch from 39a550a to 4f9089b Compare August 27, 2026 04:32
@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 Aug 28, 2026
@qiancai
qiancai merged commit 6e8d081 into pingcap:i18n-ja-release-8.5 Aug 28, 2026
4 of 9 checks passed
@ti-chi-bot ti-chi-bot Bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Aug 28, 2026
@ti-chi-bot

ti-chi-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

[LGTM Timeline notifier]

Timeline:

  • 2026-08-28 08:14:51.22502867 +0000 UTC m=+835126.396122797: ☑️ agreed by qiancai.

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. needs-1-more-lgtm Indicates a PR needs 1 more LGTM. size/XXL Denotes a PR that changes 1000+ 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