[GitHub] [spark] cloud-fan commented on a diff in pull request #42931: [SPARK-45137][CONNECT] Support map/array parameters in parameterized `sql()`

2023-09-17 Thread via GitHub
cloud-fan commented on code in PR #42931: URL: https://github.com/apache/spark/pull/42931#discussion_r1328222454 ## connector/connect/common/src/main/protobuf/spark/connect/commands.proto: ## @@ -65,6 +65,12 @@ message SqlCommand { // (Optional) A sequence of literal expres

[GitHub] [spark] cloud-fan commented on a diff in pull request #42931: [SPARK-45137][CONNECT] Support map/array parameters in parameterized `sql()`

2023-09-17 Thread via GitHub
cloud-fan commented on code in PR #42931: URL: https://github.com/apache/spark/pull/42931#discussion_r132837 ## connector/connect/common/src/main/protobuf/spark/connect/commands.proto: ## @@ -65,6 +65,12 @@ message SqlCommand { // (Optional) A sequence of literal expres

[GitHub] [spark] cloud-fan commented on a diff in pull request #42931: [SPARK-45137][CONNECT] Support map/array parameters in parameterized `sql()`

2023-09-17 Thread via GitHub
cloud-fan commented on code in PR #42931: URL: https://github.com/apache/spark/pull/42931#discussion_r1328222151 ## connector/connect/common/src/main/protobuf/spark/connect/commands.proto: ## @@ -65,6 +65,12 @@ message SqlCommand { // (Optional) A sequence of literal expres