-
Notifications
You must be signed in to change notification settings - Fork 760
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
feat: support prql #14922
feat: support prql #14922
Conversation
could we support this inside the ast crate ? |
Ast crate use tokens as parameter now, but prql's token not same as databend sql’s token. so if we want to support this inside ast, maybe we need make string as ast parser's parameter? |
I see now, this pr's approach is ok. |
Good to me, let's add some tests. |
done |
Thank you! |
Hi from the PRQL team! Excited to see this. If you'd like to add links to Databend from our website & docs, please feel free to. We did this for ClickHouse when they started supporting PRQL, so we could pretty much copy that PR.. |
@ncuwaln Could you send a pr to improve the PRQL's website? BTW @max-sixty we will write a blog about PRQL with Databend soon. |
Of course, i will send a pr soon |
I will send a pr after databend doc ready |
document cc @soyeric128 |
Already send a pr to fix it |
A blog about databend & prql: https://www.databend.com/blog/2024-04-03-databend-integrates-prql/ |
FYI PRQL/prql#4494 adds Databend to PRQL's website |
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
support prql in databend
#14811
Tests
Type of change
This change is