Skip to content

Conversation

@anuunchin
Copy link
Contributor

@anuunchin anuunchin commented Nov 21, 2025

This PR is an attempt to resolve the issue where sqlglot is not able to annotate the type of a column.

Root cause: In some cases, the selects of the expression passed to the compute_columns_schema in dlt.dataset.lineage.py include bare Identifiers. qualify() fails to qualify these bare Identifier nodes. Then annotate_types() fails downstream because it's working with unqualified columns.

Resolves #3345

@anuunchin anuunchin requested a review from zilto November 21, 2025 16:45
@anuunchin anuunchin force-pushed the fix/3345-missing-columns branch from b63f151 to 58cae6d Compare November 21, 2025 16:49
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs 079bb59 Commit Preview URL

Branch Preview URL
Nov 21 2025, 06:22 PM

@anuunchin anuunchin removed the request for review from zilto November 21, 2025 16:54
@anuunchin anuunchin force-pushed the fix/3345-missing-columns branch 2 times, most recently from 57d3bdd to b0aafc2 Compare November 21, 2025 17:51
@anuunchin anuunchin self-assigned this Nov 21, 2025
@anuunchin anuunchin force-pushed the fix/3345-missing-columns branch from b0aafc2 to 06657aa Compare November 21, 2025 17:59
@anuunchin anuunchin requested a review from zilto November 21, 2025 17:59
@anuunchin anuunchin force-pushed the fix/3345-missing-columns branch from 06657aa to 591e5be Compare November 21, 2025 18:07
@anuunchin anuunchin force-pushed the fix/3345-missing-columns branch from 591e5be to 079bb59 Compare November 21, 2025 18:13
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.

bug: missing column with SqlModel

2 participants