thswlsqls commented on PR #39102: URL: https://github.com/apache/shardingsphere/pull/39102#issuecomment-5143248454
Thanks for the detailed review. Agreed on the grammar point — routing it through `unreservedWord` also admits `SYSTEM_USER` as a table or owner name and as `SYSTEM_USER()`, which SQL-92 does not define, so I'll narrow it to the bare value specification. On the quoted case, `ColumnSegmentBinder` matches on the unwrapped identifier value and never inspects the quote kind, so `SELECT t."USER"` already skips binding on master today across every dialect that defines this set — that distinction looks binder-level rather than SQL92-specific. Happy to split it into a separate issue, or to handle it here if you prefer. -- 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]
