raminqaf commented on PR #27846: URL: https://github.com/apache/flink/pull/27846#issuecomment-4154685335
### Code review Found 2 issues: 1. **Syntax error: extra closing bracket `]` in ALTER MATERIALIZED TABLE distribution definition.** The `<distribution>` syntax block in the ALTER section has a stray `]` after the column list parenthesis, making the documented syntax invalid. The CREATE section does not have this error. https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content/docs/sql/materialized-table/statements.md#L425-L429 Same issue in the Chinese version: https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content.zh/docs/sql/materialized-table/statements.md#L426-L430 2. **Typo: "MyMateralizedTable" should be "MyMaterializedTable"** (missing "i" in "Materialized"). This appears consistently across all ADD, MODIFY, and DROP SQL examples in both the English and Chinese documentation files (14 occurrences total). https://github.com/apache/flink/blob/22e980d8fbe5f8eb04d845b0d31585c0b6f703c5/docs/content/docs/sql/materialized-table/statements.md#L443-L445 🤖 Generated with [Claude Code](https://claude.ai/code) <sub>- If this code review was useful, please react with 👍. Otherwise, react with 👎.</sub> -- 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]
