Skip to content

[fix] correct malformed JSON and doubled GET in get-ddl-stmt-action doc#3684

Merged
morningman merged 2 commits into
apache:masterfrom
boluor:fix-get-ddl-stmt-action-doc
May 20, 2026
Merged

[fix] correct malformed JSON and doubled GET in get-ddl-stmt-action doc#3684
morningman merged 2 commits into
apache:masterfrom
boluor:fix-get-ddl-stmt-action-doc

Conversation

@boluor
Copy link
Copy Markdown
Contributor

@boluor boluor commented May 19, 2026

Summary

Two errors in the FE get-ddl-stmt-action API doc (admin-manual/open-api/fe-http/get-ddl-stmt-action.md):

  • [typo](doc)fix be config buffer_pool_limit default value #25 — In the Response JSON example, the create_table array value was missing its closing double-quote: ["CREATE TABLE \tbl1` ...]["CREATE TABLE `tbl1` ..."]. The sibling create_partition/create_rollup` values are correctly quoted, confirming this is a malformation.
  • fix-document #26 — The example request line had a doubled GET: GET GET /api/_get_ddl?...GET /api/_get_ddl?....

Applied across all 12 affected versions (EN + ZH, next/4.x/3.x/2.1/2.0/1.2). The bug text was identical in every version.

Test plan

  • Verified the malformed JSON quote against the correctly-quoted sibling fields
  • Verified the doubled GET

Two errors in the get-ddl-stmt-action API doc:
- The `create_table` value in the Response JSON example was missing its
  closing double-quote (`["CREATE TABLE ...]` -> `["CREATE TABLE ..."]`).
- The example request line had a doubled `GET` (`GET GET /api/_get_ddl`).

Applied across all affected EN and Chinese versions.

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 98d79e9 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