Cory Grinstead

we should probably hold off on this until we do https://github.com/GlareDB/glaredb/issues/2490. That issue will likely result in a large refactor of our function registry anyways, so it'd probably be easiest...

> This follows postgres naming conventions. Case-sensitive column names need to be quoted. while case sensitivity can be enforced by the use of quotes, it isn't the default. this is...

labelling 'perf' as this means that we need to store and check validity bitmaps in (arrow, lance) or nullability definition levels in (parquet, delta).

> Why do we need to put a warning on CREATE CREDENTIAL? that's the point. It shouldn't show a warning, but it does.

@greyscaled do you have an example of the parquet file you used for this?

hmm, yeah i'm not seeing the same error, but a different one ``` exceptions.ExecutionException: Error during planning: Unable to fetch table provider for 'test': Not a Delta table: no log...

initial functionality is implemented in #2344 follow up issues - [ ] #2346 - [ ] #2345 - [ ] #2347 - [ ] #2348 - [ ] #2349

was looking into this a bit. I think the tricky part is that datafusion's udfs only pass the column refs, but not a schema, so we don't actually have a...

so one caveat with enabling this, Clap doesn't hide subcommands annotated with `hide`, so it does result in us generating completions for **all** commands.

note: it works it i set force_refresh to true `SET force_catalog_refresh TO true;`, but I'd expect this to work by default.