Taehoon Moon
Taehoon Moon
Thanks, I didn't aware of this, currently only a single identifier is allowed. It looks good to add support.
blocked by #417
I'm also familiar with MySQL's `AUTO_INCREMENT` syntax. There can be something to consider about the implementation plan. I'm not sure that current `Store` trait methods are enough to support `AUTO_INCREMENT`....
@KyGost In the case of MySQL syntax, we might be able to use `CREATE TABLE Test (id INTEGER AUTO_INCREMENT NOT NULL)` without using `CONSTRAINT` keyword ```rust .. ColumnOptionDef { name:...
And the design I briefly thought was something like this. Adding optional store trait ```rust pub trait AutoIncrement { fn generate_id() -> T; } ``` and implementation idea for `SledStorage`...
@KyGost 🚀 https://github.com/gluesql/gluesql/pull/160 🚀
@MRGRAVITY817 Thanks 👍
> Should we test the contents of created spool_file ? It would be good to have one, but currently... it is not easy to make test based on rustyline. We...
e.g. ```sql /* Supported */ SUM(id) COUNT(*) MAX(col), MIN(ta.col2) /* To be supported */ SUM(id + 1) MAX(col + col2) SUM(id * rate) ```
I barely remember. You were trying to insert blob, but there's no feature supported so you might use `sqlparser` directly? or used text.