Skip to content

Conversation

@PhilipTrauner
Copy link

CTE handling differs slighlty between SELECT and other kinds of statements, as SELECT grammar uses a
common_table_stmt token, while other DML statements use a with_clause token

Jille and others added 12 commits April 4, 2024 23:54
Fixed resolving refs to CTEs by adding CTEs to the aliasMap and indexing
its columns when resolving catalog references.

Fix sqlc-dev#3219
I've checked each of them, and it seems correct those fields lose their nullability
CTE handling differs slighlty between SELECT and other kinds of statements, as SELECT grammar uses a
`common_table_stmt` token, while other DML statements use a `with_clause` token
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Nov 6, 2025
@dosubot dosubot bot added the 🔧 golang label Nov 6, 2025
@PhilipTrauner
Copy link
Author

inadvertently opened here, wanted to open it here instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files. 🔧 golang

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants