iffyio commented on PR #1470: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1470#issuecomment-2410443601
Hi @emilsivervik Sure! Could you take a look at the [CLUSTERED INDEX impl](https://github.com/apache/datafusion-sqlparser-rs/blob/6cfa42c3bd0a7f1d084af001b99845e05ea3b5da/src/parser/mod.rs#L6781-L6786) which has similar syntax and likely solution for parsing and representing the `(id ASC)` type scenarios? I'm thinking we'd probably be able to reuse that type, for example if we extend the `IndexType` parsing and representation to include: ```rust pub enum IndexType { BTree(Option<Vec<ClusteredIndex>>), Hash(Option<Vec<ClusteredIndex>>), } ``` And then something similar for the KEY support. Let me know if you have any questions. Also, from the tests I'm assuming this is a mysql syntax? could you include a link to the docs maybe in the PR description where this syntax is described? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
