AfterShip / clickhouse-sql-parser

57%

master: 57%

LAST BUILD ON BRANCH codex/fix-modify-column-modifiers

branch: codex/fix-modify-column-modifiers

CHANGE BRANCH

x

Reset

Sync Branches

• codex/fix-modify-column-modifiers
• 14-2
• add-benchmarks
• add-coverall
• add-modify-ttl
• add-more-badges
• add-valid-until-clause
• ajruckman_alter_table_delete_where
• ajruckman_placeholder_and_select_fix
• alert-autofix-1
• allow-string-as-tablename
• allow-var-start-with-dollar
• alter-role-settings
• alter-table-drop-detached
• auto-version
• case-statement-when-spacing
• change-sql-string-to-compact
• claude/end-not-reserved
• claude/frosty-wiles-784a80
• claude/issue-184-20250827-0341
• claude/romantic-kowalevski-5a7ec1
• codec-delta-level
• codex/fix-contextual-keyword-parsing
• codex/fix-expression-precedence
• codex/fix-lexical-fidelity
• codex/fix-named-type-token-errors
• codex/fix-parser-token-errors
• codex/fix-semantic-traversal
• codex/source-spans-diagnostics
• column-value-without-type
• copilot/add-beautify-format-alter-role
• copilot/add-beautify-format-for-create-user
• copilot/add-beautify-format-instruction
• copilot/add-beautify-format-support
• copilot/add-ident-break-for-insert
• copilot/add-ident-break-for-settings
• copilot/add-ident-break-from-expression
• copilot/add-ident-break-on-cluster
• copilot/add-support-in-partition-expression
• copilot/fix-12
• copilot/fix-140
• copilot/fix-179
• copilot/fix-181
• copilot/fix-187
• copilot/fix-75186bce-1dd8-4e67-bcd7-685469aa2574
• copilot/fix-e5e3e050-4a0c-4226-8f60-806cafedacba
• copilot/fix-stderr-routing-issue
• copilot/refactor-formatter-chaining-methods
• copilot/update-tool-version-to-0417
• dev
• dev1
• distinct-on
• feat/add-order-by-with-fill-support
• feat/alter_modify_query_support
• feat/lagr-parser-gaps
• feat_support_db_comment
• feature/access-tuple-with-index
• feature/add-projection
• feature/add-union-distinct
• feature/adjust_engineExpr_String_printing_order
• feature/allow-select-modifier
• feature/alter-table-replace
• feature/check-expression
• feature/clear-projection-statement
• feature/column-def-alias-materialized
• feature/create-function
• feature/create-or-replace
• feature/create-role
• feature/delete-from
• feature/drop-projection
• feature/drop-user-role
• feature/grant-privilege
• feature/grouping-sets
• feature/insert-expr
• feature/insert-with-type-column
• feature/materialize-index-and-projection
• feature/optimize-expression
• feature/query-parameter
• feature/rename-table
• feature/select-in-function
• feature/support-json-type-skip
• feature/support-placeholder
• feature/support-ttl-policy
• feature/support-tuple
• feature/timestamp-interval
• fix-array-join-take-2
• fix-bugs-1
• fix-cannot-parse-join-only
• fix-database-column-name
• fix-describe-table
• fix-global-join-modifier
• fix-interval-column-name
• fix-materialized-bug
• fix-no-as-alias
• fix-order-by-panic
• fix-panic-in-eof-error
• fix-parenthesized-set-operator-left-operand
• fix-select-limit-add-select-final
• fix-table-function-arg-exprs
• fix/allow-parameters-for-null-engine
• fix/alter-table-add-index-settings
• fix/alter-table-modify
• fix/alter-table-settings-statement-level
• fix/ast-walker
• fix/between-expr
• fix/between-visitor
• fix/cast-expr
• fix/cast-expr-operand
• fix/cast-in-table-function-arg
• fix/column-alias-string
• fix/create-dictionary-format
• fix/distinct-keyword-preservation
• fix/distinct-on-dot-notation
• fix/dollar-sign
• fix/dont_stop_walk_nil
• fix/double-quote-as-ident
• fix/drop-database-sync
• fix/expression-precedence
• fix/format-function
• fix/group-by-column-with-keyworld
• fix/insert-into
• fix/issue-118
• fix/issue-176
• fix/issue-187
• fix/issue-222
• fix/issues-108
• fix/keyword-arg-functions
• fix/keyword-as-column-in-select
• fix/keyword_alias
• fix/lexer-numbers-comments-utf8
• fix/lexer-unterminated-comment-hang
• fix/lowercase-array-join
• fix/modify-ttl
• fix/multi-join
• fix/multi-line-comments
• fix/multi_distinct_except
• fix/nil-deref-error-formatting
• fix/npe-unexpected-token
• fix/panic-invalid-sql
• fix/parse-comment-literal
• fix/position-tracking
• fix/readme
• fix/remove-extra-leading-space
• fix/reserved-keyword-split
• fix/select-as-query
• fix/select-issues
• fix/show-access
• fix/signed-number-after-bracket
• fix/string-as-table-ident
• fix/surface-lexer-errors
• fix/system-reload-on-cluster
• fix/table-keyword-alias-upstream
• fix/table-visitor
• fix/trailing-operator-at-eof
• fix/traversal-drift
• fix/union-all
• fix/walk_when_node
• fix/when_case_then
• fix/window_clause
• fix/wraperror-index-out-of-range
• fix/wrong-unary-expr-string
• ft-enhance-json-options
• improve-lexer
• improve-rename-syntax
• improve/parse-expr
• issue-204
• issue-23
• json-type-hints
• keyword-ident-expression-position
• lexer-naming
• lexer-string-escapes
• master
• minor-refactor-column-type
• patch-1
• project-group-or-order
• public-api-additions
• refactor-join-modifiers
• refactor/allow-to-use-precedence
• refactor/expect-token-kind
• refactor/operation-name
• replicated-database-fix
• reserved-keyword-split
• role/alter
• role/on-cluster-expr
• select-format
• select-with-left-join
• settings-bool-parser
• sharad/add-comment-support-to-create-view
• sharad/bug_fix
• sharad/codec_parser
• sharad/codec_parser_v3
• sharad/column-modifier-order
• sharad/delta_number_literal
• sharad/dictionary_v1
• sharad/fix-mv-comment-before-as
• sharad/fix-rmv-column-list-engine
• sharad/mv_definer
• sharad/named_parameter_v1
• sharad/row-policy-ddl
• sharad/table_comment
• structured-parse-errors
• support-concat-operator
• support-function-table-settings
• support-multiple-array-joins
• support-nested-tuple-definitions
• support-offset-without-limit
• support-refresh-in-mv
• support/table-alias-without-as
• swgillespie/alter-table-segv
• swgillespie/clickhouse-remotesecure
• test-automation
• test-lowercase-array-join-ast
• traversal-drift-test
• understand-gcd-codec
• unify-sql-terms
• update-readme

Build # 40629571622

Build Type

Pull #338

github

Committed by git-hulk

Commit Message

Fix MODIFY COLUMN parsing when the data type is omitted

ALTER TABLE ... MODIFY COLUMN can reject valid modifier-only changes or
store a modifier as the column type. Modifier keywords are treated as
type names when the declaration omits its data type.

For example, CODEC(LZ4) is stored in Column.Type instead of Column.Codec,
while a leading COMMENT causes parsing to fail. This bug predates #337.

Recognize COMMENT, CODEC and TTL as modifiers when the type is omitted,
preserving their AST fields through formatting and reparsing. Preserve
explicit-type parsing and duplicate-modifier rejection, and regenerate
the AST and formatting fixtures for the affected ALTER statements.

Assistant-By GPT-6

Pull Request Pull Request #338: Fix MODIFY COLUMN parsing when the data type is omitted

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

9838 of 17385 relevant lines covered (56.59%)

3681.86 hits per line