Replies: 3 comments 1 reply
-
5th option: allow both options 3 and 4
and potentially (???)
|
Beta Was this translation helpful? Give feedback.
-
If you voted, please vote again as I added a fifth option |
Beta Was this translation helpful? Give feedback.
-
@gforsyth read my mind with his suggestion, but I have one thought: do you think it might be confusing to people that it's possible to specify something which isn't a database in the database argument? I like the keyword It's certainly unfortunate that both meanings of schema are also so widespread, if the column schema was only an Ibis thing I'd definitely vote to rename it just to columns. Maybe To summarize, I'm not proposing additional options, rather amendments to ones already proposed
|
Beta Was this translation helpful? Give feedback.
-
We've made a lot of progress on cleaning up specifying
database
andschema
(the hierarchy kind) in all the places in can be used.Users can
list_*
inside schemas and databases where supported and they can also read tables across schemas and databases where supported.There's one remaining major API where we don't have a clear way to do this:
create_table
.It's a bit unfortunate, but we have an argument named
schema
that is for specifying column names and types, not a namespace inside of database.There are a few options here.
Rename
create_table
's currentschema
argument tocolumns
We'd repurpose
schema
for hierarchies.database
is unchanged.This is likely to cause breakage and probably confusion since the same word now means something very different.
Keep
schema
, replacedatabase
withcatalog
;database
becomesschema
This is a pretty nuclear option, as we'd want to change everything else for consistency.
Keep
database
as is, and use it for specifying namespacesDeprecate
database
and use dotted path syntax in the table nameMight be some annoying issues with quoting and backend specificity, but I think that can largely be punted to sqlglot.
If we do this we should probably support
con.table("db.schema.table")
as well.Let's vote on the options!
I'm happy to add other options as they come up.
4 votes ·
Beta Was this translation helpful? Give feedback.
All reactions