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.

Sign up for GitHub

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

Jump to bottom

sql: unify `daft.catalog` and `daft.sql.catalog` #3586

Open

universalmind303 opened this issue Dec 16, 2024 · 0 comments

Labels

sql

Contributor

universalmind303 commented Dec 16, 2024 •

edited

Loading

WRT to the catalogs, @universalmind303 what do you think of starting to unify around the DaftMetaCatalog that I introduced in #3036?

I think we have a few competing standards atm (including the SQLCatalog). It could be good to start having a catalog abstraction that can be shared across our different frontends

yes, that is something I want to do and have been thinking about. I'll open up an issue to unify daft.catalog and daft.sql.catalog

Originally posted by @universalmind303 in #3581 (comment)

The text was updated successfully, but these errors were encountered:

universalmind303 changed the title ~~unify daft.catalog and daft.sql.catalog~~ sql: unify daft.catalog and daft.sql.catalog

universalmind303 added the sql label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment