WeblWabl

> Here is a list that has not been done yet, for interested contributor to better view: > > * [ ] GetIndexedField > * [ ] ScalarVariable > *...

Sounds good. I have never contributed to this project before but am interested in helping out where I can and offering my support for this project -- @alamb ~~if I...

> Hi @devanbenz , I think `ScalarVariable` has not been done yet and you could have a try, though it may not be possible.🤔 I think maybe it should be...

> Here are some thoughts to help people figure out what SQL matches to what expression: > > * `GetIndexedField` --> `SELECT col['field_name'] > * ScalarVariable --> Not sure (maybe...

@alamb if you could please take a look at https://github.com/sqlparser-rs/sqlparser-rs/pull/1260 -- it's linked to this issue :)

> > @alamb if you could please take a look at [sqlparser-rs/sqlparser-rs#1260](https://github.com/sqlparser-rs/sqlparser-rs/pull/1260) -- it's linked to this issue :) > > Thanks @devanbenz -- can you explain how this is...

> It seems to me that `Expr::ScalarVariable` currently is parsed from `ast::Identifier` (that starts with `@`)-- perhaps instead of adding a new ast node, you could just create an equivalent...

Is this issue still prevalent? Would love to take it on if so.

@alamb while working on https://github.com/apache/datafusion/issues/9726 I was running in to this. I often use MSSQL for work so the ScalarVariable syntax that I'm used to is `DECLARE @foo AS INT;`...

@alamb or @iffyio https://github.com/sqlparser-rs/sqlparser-rs/pull/1235 please take a look when you can. Also, please let me know if there's anything to improve. I'm a tad bit newer to Rust so always...