hdimer opened a new pull request, #2418: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/2418
Fixes #2409. A bracket-quoted identifier whose value contains `]` (e.g. `[a]]b]`, value `a]b`) serialized back to `[a]b]`, which no longer re-parses. This doubles each `]` on display, mirroring the tokenizer which folds `]]` into `]`. Redshift also parses nested quoted identifiers like `["a]b"]`, whose value is stored as a complete double-quoted string (`"a]b"`) with a literal inner `]`. The AST doesn't distinguish that from a plain bracket ident, so a value that is a complete `"..."` string is left unchanged; otherwise the `]` are doubled. Added a round-trip test; the existing Redshift nested-identifier test covers the verbatim case. Used AI assistance on this; I reviewed and tested it. -- 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]
