-
Notifications
You must be signed in to change notification settings - Fork 48
sqlglot.errors.ParseError for sample from website #315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hi @samiabboud, could you help check which version of |
Thank you for the quick response @ashleyxuu . Here are the versions I have installed (as part of bigframes latest I believe).
|
Upgrading sqlglot to 20.8.0 doesn't resolve the error. Upgrading ibis-framework results in the following dependency error:
|
+1 |
Could you try upgrading bigframes to the latest version
|
I already updated these modules but it was not resolved. |
https://partner.cloudskillsboost.google/course_sessions/11372701/labs/448513 same issue in this cloudskill lab with colab enterprise |
+1 |
2 similar comments
+1 |
+1 |
We're working on fixing it. Internal issue number: 326126888 |
Hi, @ashleyxuu Code:
Error:
I am waiting for fix. Thanks. |
@uysalfurkan Thank you so much for reporting this issue! I apologize for the delay in responding. To help me track down this dependency mismatch, could you please provide the full call stack? Also, to reproduce the error in our side, would you mind sharing your environment details? You can generate these using the following command:
|
In addition to #315 (comment) could you also please run the following cell:
This will help us confirm that the version of bigframes in the notebook aligns with that claimed by |
@chelsea-lin , @tswast sorry for delayed response. Environment details: Full call stack:
|
Re-opening, as it appears this issue occurs even with latest supported sqlglot. |
A thought: ibis 7.x and 8.x only use sqlglot in the BigQuery backend for unnest support. This isn't currently used by BigQuery DataFrames. We might be able to monkeypatch out the problematic code to avoid this. |
I'd also like to try updating to ibis 8.x in #277 to see if that fixes this issue. |
I'm having a similar issue even with basic operations like bq_df.head(). This is all taking place in the BigQuery notebook interface with the default runtime. Here's the full call stack:
|
@ZeroCool2u Thanks for the report. My teammate @chelsea-lin was able to determine that there is a bug in sqlglot's parsing of BigQuery table IDs, which has been reported and hopefully fixed in a future release. In the meantime, I believe bigframes 0.22.0 will have worked around this issue. Could you please try with that version and report back if it is fixed? In a notebook:
And then restart your notebook runtime. |
@tswast @chelsea-lin Just tested and seems fixed! Thank you!! |
Hello everyone,
Running a copy/pasted sample from the site is raising a
sqlglot.errors.ParseError
. Version issues maybe? Please see details below.Your help is appreciated!
Cheers,
Sami
Environment details
python --version
3.9.18pip --version
pip 23.0.1bigframes
version:pip show bigframes
0.19.0Steps to reproduce
Code example
Stack trace
The text was updated successfully, but these errors were encountered: