[GitHub] [spark] xkrogen commented on pull request #38864: [SPARK-41271][SQL] Support parameterized SQL queries by `sql()`

2022-12-06 Thread GitBox
xkrogen commented on PR #38864: URL: https://github.com/apache/spark/pull/38864#issuecomment-1339991238 Regarding supporting two syntaxes vs. standardizing on one, I don't have much of an opinion, but I would be curious about how Spark has handled such decisions in the past when there are c

[GitHub] [spark] xkrogen commented on pull request #38864: [SPARK-41271][SQL] Support parameterized SQL queries by `sql()`

2022-12-05 Thread GitBox
xkrogen commented on PR #38864: URL: https://github.com/apache/spark/pull/38864#issuecomment-1338091122 I managed to find a better SQL standard reference in the form of _SQL: The Complete Reference_ (2003), which has an entire chapter devoted to Dynamic SQL (beginning from page 547). You ar

[GitHub] [spark] xkrogen commented on pull request #38864: [SPARK-41271][SQL] Support parameterized SQL queries by `sql()`

2022-12-02 Thread GitBox
xkrogen commented on PR #38864: URL: https://github.com/apache/spark/pull/38864#issuecomment-1335865990 What is the relationship between this PR and #38712? Why do we have two PRs? If this PR is superceding #38712, can we continue discussion here on which identifier to use based on [m