Don't try to create a table if it already exists

[zmitchell]

I think a nice quality of life improvement would be to add an option to change the behavior of MyTable.create() from

CREATE TABLE MyTable

to

CREATE TABLE MyTable IF NOT EXISTS

I looked at the source and I couldn't actually find where MyTable.create() is defined. Maybe it's because I'm not well-versed in macros yet.

[Ygg01]

Yeah, as far as I saw, all tables support this code.

The MyTable.create is created by a macro so I believe this is the part that's creating the table.

[antoyo]

As far as I remember (it's been two years :) ), I first chose the IF NOT EXISTS behaviour but then switched to removing it because you can simply choose to ignore the error instead of using ? or unwrap().
Of course, other errors can happen, so maybe you still find more convenient to have IF NOT EXISTS.
Please tell me your thoughts on this.

[michaeldoubek]

@antoyo I would like to help, but this would be my first experience in open source. Do you have some guidelines or can you advice me via some fast channel if you struggle? Thx.

[antoyo]

So, codegen generation happens here.
There's a file for the postgres codegen and another for the sqlite codegen.
The CREATE TABLE is generated here.

I'm often on #rust on irc.gnome.org if you want to talk.

Thanks for proposing your help.