Skip to content

[fix] correct localization and example errors in open-api & system-tables docs#3689

Merged
morningman merged 2 commits into
apache:masterfrom
boluor:fix-open-api-system-tables-batch
May 20, 2026
Merged

[fix] correct localization and example errors in open-api & system-tables docs#3689
morningman merged 2 commits into
apache:masterfrom
boluor:fix-open-api-system-tables-batch

Conversation

@boluor
Copy link
Copy Markdown
Contributor

@boluor boluor commented May 19, 2026

Summary

A batch of doc-audit fixes (findings #32-#38) across the open-api (fe-http) and system-tables docs:

  • [feature] Add OSS upload & PWA support #32system-tables/information_schema/table_privileges.md: frontmatter language was zh-CN for an all-English doc → en.
  • [CDN] Upload files to OSS #33open-api/fe-http/statistic-action.md: untranslated Chinese (frontmatter description, ## Description body, parameter values) → English.
  • [doc](add)add show query profile doc #34open-api/fe-http/statement-execution-action.md: untranslated Chinese translated; the description referred to a sql field but the request body actually uses stmt → corrected.
  • [PWA] Add PWA support #35 — same file: the request path POST /api/query/<ns_name>/<db_name> has two path params but only <db_name> was documented → added a <ns_name> entry.
  • Change static files to CDN #36open-api/fe-http/show-meta-info-action.md: Chinese in the Path parameters section → None.
  • Change static files to CDN #37open-api/fe-http/upload-action.md: the Response JSON example was missing a comma after the absPath field (invalid JSON) → comma added. Fixed in both EN and ZH copies since it is a code-syntax error.
  • Revert "Change static files to CDN" #38system-tables/information_schema/column_statistics.md: broken anchor statistics#viewing-statistics#viewing-column-statistics (only the next/4.x versions, whose target heading was renamed; 2.1/3.x targets still have the old anchor and are left unchanged).

The localization fixes (#32-#36) apply only to the English doc tree; the Chinese i18n/zh-CN copies are correctly Chinese and left as-is.

Test plan

  • Verified each doc body / request spec to confirm the correct values (e.g. stmt field name, with the <ns_name> param)
  • Verified no Chinese remains in the translated English-tree docs
  • Verified the #viewing-column-statistics anchor exists in the next/4.x target doc, and the old anchor still exists in 2.1/3.x

…bles docs

Batch of fixes in the open-api (fe-http) and system-tables docs:
- table_privileges.md: frontmatter `language` was `zh-CN` for an all-English doc -> `en`.
- statistic-action.md: untranslated Chinese (description, `## Description`, `无` values) -> English.
- statement-execution-action.md: untranslated Chinese translated; the description
  referred to a `sql` field but the request body uses `stmt` -> corrected; the
  undocumented `<ns_name>` path parameter is now documented.
- show-meta-info-action.md: Chinese `无` in Path parameters -> `None`.
- upload-action.md: Response JSON example was missing a comma after `absPath`
  (fixed in both EN and ZH copies, as it is a code-syntax error).
- column_statistics.md: broken anchor `#viewing-statistics` -> `#viewing-column-statistics`
  (only the next/4.x versions, whose target heading was renamed).

Localization fixes apply only to the English doc tree; the Chinese i18n copies
are left as-is.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The CI pipeline no longer builds these legacy versions, so changes there
cannot be merged. Restore those files to their base state.
@morningman morningman merged commit 46e7120 into apache:master May 20, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants