Skip to content

refactor: decouple SQL building logic from sqlglot_ir#2491

Merged
chelsea-lin merged 2 commits intomainfrom
main_chelsealin_refactorsqlglot
Mar 4, 2026
Merged

refactor: decouple SQL building logic from sqlglot_ir#2491
chelsea-lin merged 2 commits intomainfrom
main_chelsealin_refactorsqlglot

Conversation

@chelsea-lin
Copy link
Contributor

Fixes internal issue 418025765🦕

@product-auto-label product-auto-label bot added size: l Pull request size is large. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels Mar 4, 2026
@chelsea-lin chelsea-lin changed the title refactor: split sql builders from the sqlglot_ir class refactor: decouple SQL building logic from sqlglot_ir Mar 4, 2026
@chelsea-lin chelsea-lin force-pushed the main_chelsealin_refactorsqlglot branch from 3003e3f to 684e5e6 Compare March 4, 2026 18:48
@chelsea-lin chelsea-lin force-pushed the main_chelsealin_refactorsqlglot branch from 684e5e6 to 63f9710 Compare March 4, 2026 18:48
@chelsea-lin chelsea-lin requested a review from sycai March 4, 2026 18:49
@chelsea-lin chelsea-lin marked this pull request as ready for review March 4, 2026 18:49
@chelsea-lin chelsea-lin requested review from a team as code owners March 4, 2026 18:49
@chelsea-lin chelsea-lin merged commit 47a0feb into main Mar 4, 2026
22 of 25 checks passed
@chelsea-lin chelsea-lin deleted the main_chelsealin_refactorsqlglot branch March 4, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants