Alter table rename

[f0ssel]

I think this works, still need to test in dev

closes #809

[scsmithr]

This change will be a good thing to test with SLTs.

Something like

statement ok
create schema rename_tables_test;

statement ok
set search_path = rename_tables_test;

statement ok
set enable_debug_datasources to t;

statement ok
create external table t1 from debug options (table_type = 'never_ending');

statement ok
alter table t1 rename to t2;

...

[f0ssel]

Based on https://github.com/GlareDB/glaredb/actions/runs/4671477670/jobs/8272523797?pr=848, I think I need to update the proto files. @scsmithr is there a command I need to run?

nvm I see you edit the proto files and it generates the code, not the other way around. 👍

[f0ssel]

Realize I also still need to properly parse the if exists, that's currently just not happening.

[vrongmeal]

Looks good 💯

Just the one comment about returning error when removing from entries. @scsmithr how do you feel about that?

[f0ssel]

@scsmithr it seems that the default alter table parse doesn't seem to give me an if_exists option, so I'd need to do my own parsing similar to here I think - https://github.com/GlareDB/glaredb/blob/f0ssel/alter-table-rename/crates/sqlexec/src/parser.rs#L201 . Maybe we just scrap the if exists option for now?

made changes

[scsmithr]

@scsmithr it seems that the default alter table parse doesn't seem to give me an if_exists option, so I'd need to do my own parsing similar to here I think - https://github.com/GlareDB/glaredb/blob/f0ssel/alter-table-rename/crates/sqlexec/src/parser.rs#L201 . Maybe we just scrap the if exists option for now?

Let's skip and open an issue for it. It should be easy to add to the upstream lib.