Iñigo Hidalgo
06/19/2023, 4:33 PMpd.read_sql
would do.
The dataset implementation is here, and I wrote a tiny example pipeline here.
I would appreciate any feedback you could give, as an issue on the project or a response through here :)datajoely
06/19/2023, 4:35 PMNok Lam Chan
06/19/2023, 5:06 PMJuan Luis
06/20/2023, 11:01 AMIñigo Hidalgo
06/20/2023, 11:16 AM> I think the more solid path forward would be to say "all things SQL in Kedro go through Ibis" and make it a "first level" optional dependency. No need to add more levels of indirection I'd say. To confidently commit to that plan, I propose we put this initial dataset in the hands of users and play around with it a bit more.
>
@Juan LuisWould definitely be cautious about that, if other users are like us they probably have a lot of written SQL queries I don’t think they’d want to migrate to Ibis directly 😅 Though promoting it as a more robust and flexible alternative would probably do the job
Juan Luis
06/20/2023, 11:23 AMMarc Gris
06/29/2023, 12:08 PMCody Peterson
07/10/2023, 5:28 PMWould definitely be cautious about that, if other users are like us they probably have a lot of written SQL queries I don’t think they’d want to migrate to Ibis directlywe agree! Ibis has a `.sql()` method on tables to act as an escape hatch and enable going back and forth between Ibis and SQL easy (with a complementary
<http://ibis.to|ibis.to>_sql()
method). it also accepts a SQL dialect and uses sqlglot under the hoodJuan Luis
07/10/2023, 7:17 PM