juntaozhang commented on PR #4318: URL: https://github.com/apache/calcite/pull/4318#issuecomment-2816315853
> I don't see this in the new tests I know your concerns, le't me clarify it, just as the conclusion of our discussion at the end in JIRAļ¼ ``` 2. The toSqlString method should not return an unparsable and nonstandard result to users, the parentheses need be removed from the <table argument>. ``` So the exsiting cases only need to modify to cover the PR, because we don't change the parse logic, so new case is not needed. If you think the named parameter syntax need to enrich the test cases, in this point I am happy add it in the PR. -- 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]
