gluesql icon indicating copy to clipboard operation
gluesql copied to clipboard

implement show indexes from table

Open ghost opened this issue 3 years ago • 6 comments

implement show index from table syntax

ghost avatar Apr 12 '22 14:04 ghost

I don't see a way for me to assign myself this, but I'll volunteer to take a look and see if I can implement it.

ghost avatar Apr 12 '22 14:04 ghost

Is it correct to support the syntax?

  • SHOW INDEXES FROM table_name;

assigned to you. thanks for the report

ever0de avatar Apr 12 '22 14:04 ever0de

Is it correct to support the syntax?

* `SHOW INDEXES FROM table_name;`

assigned to you. thanks for the report

Yes, you are correct. I'll try to fix the title.

ghost avatar Apr 12 '22 14:04 ghost

If I'm viewing sqlparser correctly, it doesn't support show indexes syntax, so I put in an issues with the sqlparser team requesting that enhancement.

ghost avatar Apr 12 '22 16:04 ghost

In sqlparser-rs version 0.16, this syntax is supported:

ql: "show indexes from mytable" AST: [ShowVariable { variable: [Ident { value: "indexes", quote_style: None }, Ident { value: "from", quote_style: None }, Ident { value: "mytable", quote_style: None }] }]

ghost avatar Apr 17 '22 00:04 ghost

PR #554 adds a third column to the output (displays expression used to define the index). Once that PR is merged, this issue can be closed.

ghost avatar Jun 01 '22 14:06 ghost

resolved in #551

panarch avatar Apr 02 '23 06:04 panarch