raminqaf commented on PR #27846: URL: https://github.com/apache/flink/pull/27846#issuecomment-4154703700
#### Nits (lower confidence) 3. **Extra space in `[OR ALTER ]`** -- In create.md (both EN and ZH), the list item and heading read `CREATE [OR ALTER ]MATERIALIZED TABLE` with a space before the closing bracket. Should be `CREATE [OR ALTER] MATERIALIZED TABLE` for consistency with the pattern on the previous line (`[CREATE OR] REPLACE TABLE`). https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content/docs/sql/reference/ddl/create.md#L35-L37 https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content.zh/docs/sql/reference/ddl/create.md#L39-L41 4. **Redundant trailing `|` on SUSPEND and REFRESH lines** -- The ALTER syntax block has both a trailing `|` at the end of the SUSPEND/RESUME line and a leading `|` on the next line, creating a double-pipe separation. The ADD/MODIFY/DROP lines above only use leading `|` without trailing `|`. https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content/docs/sql/materialized-table/statements.md#L396-L401 5. **Missing comma before "also see" in MODIFY section** -- The DROP section correctly has `", also see"` (line 491) but the MODIFY section is missing the comma: `"also see"` (line 469). https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content/docs/sql/materialized-table/statements.md#L468-L470 🤖 Generated with [Claude Code](https://claude.ai/code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
