Please include SQL comments in ast object

[labs20]

As it is now, some proprietary SQL features are not supported, like FOR XML PATH() on SQL Server for example.

As long as it would be nice to eventually have such language support, if the ast object was at least keeping the original SQL comments, it would be possible to us to pre-parse the sql text, transform unknown tags for the parser into comments, and later when rebuilding the sql, either revert back those comments to normal tags or even do custom replacements based on some arbitrary logic.

This would make a lot easier to live with parser limitations.

Thanks!

[JounQin]

Is there any plain for it? I need location info of AST at the same time.

[cchepelov]

The underlying parser, https://github.com/pegjs/pegjs, has functionality in its main branch to return comments back into the AST. Unfortunately, it is not (yet) part of the latest released version, and it has been a while since there was activity in that repo.

The apparent successor to pegjs is https://github.com/peggyjs/peggy/

[cchepelov]

cchepelov#1