-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
SQLAlchemy source for Lumen AI #1051
Comments
Ideally, we would simply have a SQLAlchemySource so we don't have to re-invent the wheel of wrapping all the SQL dialects. |
Can you expand on how you see
|
Yes, I was thinking this route.
I don't think this matters since the LLM is the translation layer, which is why we include this in the prompt: ![]() The reason I suggested SQLAlchemy is because you can use URLs for credentials: url = (
"snowflake://<user_login_name>:<password>"
"@<account_identifier>/<database_name>"
"?warehouse=<warehouse_name>"
) Or a pydantic/parameterized model for common keywords, like here: ConnectionComponents(
driver=AsyncDriver.POSTGRESQL_ASYNCPG,
username="prefect",
password="prefect_password",
database="postgres"
) |
Include
BigQuery
as a source for Lumen AI.The text was updated successfully, but these errors were encountered: