Skip to content

Conversation

@kyleconroy
Copy link
Collaborator

  • Add PRIMARY KEY column constraint support in column definitions
    Outputs PRIMARY KEY columns as Function tuple in Columns definition

  • Validate CREATE VIEW TO clause (only valid for MATERIALIZED VIEW)
    Returns parse error for regular VIEW with TO clause

  • Fix negated literal formatting in Subquery context
    Outputs as Literal Int64_-N instead of Function negate when inside Subquery
    Preserves Function negate output for view() function arguments

Tests enabled:

  • 00098_j_union_all
  • 01668_avg_weighted_ubsan
  • 02189_join_type_conversion
  • 02554_invalid_create_view_syntax (parse_error)
  • 03532_redis_empty_variant_key

…ted literals

- Add PRIMARY KEY column constraint support in column definitions
  Outputs PRIMARY KEY columns as Function tuple in Columns definition

- Validate CREATE VIEW TO clause (only valid for MATERIALIZED VIEW)
  Returns parse error for regular VIEW with TO clause

- Fix negated literal formatting in Subquery context
  Outputs as Literal Int64_-N instead of Function negate when inside Subquery
  Preserves Function negate output for view() function arguments

Tests enabled:
- 00098_j_union_all
- 01668_avg_weighted_ubsan
- 02189_join_type_conversion
- 02554_invalid_create_view_syntax (parse_error)
- 03532_redis_empty_variant_key
- Fix inSubqueryContext handling in explainTableExpression for subqueries with aliases
- Enable test 01761_cast_to_enum_nullable (already passing)
- Enable test 01818_case_float_value_fangyc (already passing)
- Enable test 03290_limit_by_segv (already passing)
- Add AsTableFunction field to CreateQuery AST for CREATE TABLE AS function()
- Parse table function calls in CREATE TABLE AS and store in AsTableFunction
- Add SHOW CREATE TABLE output format with database/table identifiers
- Enable 2 passing tests: 02118_show_create_table_rocksdb, 03293_forbid_cluster_table_engine
@kyleconroy kyleconroy merged commit 64d3d75 into main Dec 26, 2025
1 check 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.

3 participants