feat: enable SQLGlot compiler as default#2248
Draft
+5 −3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
product-auto-labelbot added size: l 
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 7a011e1 to d634b06Compareproduct-auto-labelbot added size: xs chelsea-lin added the do not merge chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from d634b06 to c90c084Comparechelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from c90c084 to eb8e92cComparechelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch 2 times, most recently from 689b4e4 to 39096e5Compare This was referenced Nov 25, 2025
chelsea-lin added a commit that referenced this pull request Nov 26, 2025
This change aims to fix the `test_timestamp_series_diff_agg` test failing in #2248. Fixes internal issue 417774347 🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 39096e5 to 2f63b1fComparechelsea-lin added a commit that referenced this pull request Dec 1, 2025
This change aims to fix some string-related tests failing in #2248. Fixes internal issue 417774347🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 2f63b1f to 6d1b653Comparechelsea-lin added a commit that referenced this pull request Dec 16, 2025
This change aims to fix some string-related tests failing in #2248. Fixes internal issue 417774347🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 6d1b653 to 82ac904Compare This was referenced Dec 17, 2025
shuoweil pushed a commit that referenced this pull request Dec 17, 2025
This change aims to fix some string-related tests failing in #2248. Fixes internal issue 417774347 🦕
chelsea-lin added a commit that referenced this pull request Dec 19, 2025
…t _literal (#2337) This change can resolve two doctests failures in #2248: `groupby.GroupBy.rank` and `bigframes.ml.metrics.roc_curve` Fixes internal issue 417774347🦕 --------- Co-authored-by: Shenyang Cai <[email protected]>
shuoweil pushed a commit that referenced this pull request Dec 19, 2025
…t _literal (#2337) This change can resolve two doctests failures in #2248: `groupby.GroupBy.rank` and `bigframes.ml.metrics.roc_curve` Fixes internal issue 417774347🦕 --------- Co-authored-by: Shenyang Cai <[email protected]>
tswast pushed a commit that referenced this pull request Dec 19, 2025
This change can resolve three doctests failures in #2248: - Added `to_json` support to the compiler backend. - Fixed minor bugs on AI ops - Fixed minor bugs on windows compiler Fixes internal issue 417774347 🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 82ac904 to 5387f52Compare This was referenced Dec 22, 2025
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 5387f52 to 6a7dbe0Comparechelsea-lin added a commit that referenced this pull request Dec 23, 2025
This change can resolve the `bigframes.ml.metrics.roc_auc_score` doctests failures in #2248. Fixes internal issue 417774347 🦕
chelsea-lin added a commit that referenced this pull request Dec 23, 2025
) This change can resolve `read_gbq` doctests failures in #2248. Fixes internal issue 417774347🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 6a7dbe0 to cecd9c3Compare This was referenced Dec 30, 2025
chelsea-lin added a commit that referenced this pull request Jan 3, 2026
This change can resolve multiple bug fixes in #2248: - `str.extract` doctest - multiple blob ops failures - `test_geo_x_non_point` and `test_geo_drop_duplicates` Fixes internal issue 417774347 🦕 --------- Co-authored-by: Tim Sweña (Swast) <[email protected]>
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from cecd9c3 to 92626a0Comparechelsea-lin added a commit that referenced this pull request Jan 6, 2026
This change can resolve `test_series_bool_bool_operators_scalar` and `test_series_int_int_operators_scalar` presubmit failures in #2248. Fixes internal issue 417774347🦕
chelsea-lin added a commit that referenced this pull request Jan 7, 2026
This change can resolve the following bugs in #2248: - fix `test_isin_*` tests failures. - fix `test_*explode*` tests failures. - fix `test_geo_x_non_point` tests failure. Fixes internal issue 417774347🦕
chelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from 92626a0 to d132cd9Comparechelsea-linforce-pushed the main_chelsealin_enablesqlglot2 branch from d132cd9 to 3972dc7CompareSign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes internal issue 417774347🦕