Phillip Cloud
Phillip Cloud
I'm not sure we should do this until we know it's actually an issue after we fix the excessive inlining.
@NickCrews If you have some concrete numbers that show a reliable performance improvement by altering the SQL here in some way, please post them here.
This PR does not have a description that matches what it does nor describes why it's necessary. The TOML change also doesn't have any stated motivation.
Closing this for now. We can't accept this PR as is: 1. The description doesn't match the changes. 2. Loading the httpfs extension is not something we should do at...
It seems like this is something that many folks are wanting/asking about. I think we should try to include this in the 10.0.0 release.
Closed by #1648
@machow We worked around this broken behavior in ibis in https://github.com/ibis-project/ibis/pull/5741 by effectively monkey patching `dialect.normalize_name`. Yeah, it's hack, but it's not clear if anyone is fixing bugs in this...
Since this behavior is fundamentally broken, we had to work around this in ibis in https://github.com/ibis-project/ibis/pull/5741. If you use ibis, we do the minimum amount of munging need to make...
We had to solve this in https://github.com/ibis-project/ibis and it's not [particularly bad](https://github.com/ibis-project/ibis/blob/master/ibis/expr/types/pretty.py#L229-L241). You don't need two calls: you can execute a limit with `N+1` rows where `N` is the number...
SQLAlchemy 2.0 was [released yesterday](https://www.sqlalchemy.org/blog/2023/01/26/sqlalchemy-2.0.0-released/)!