subject:"\[GitHub\] \[spark\] MaxGekk commented on a diff in pull request #42752\: \[SPARK\-45033\]\[SQL\] Support maps by parameterized `sql\(\)`"

[GitHub] [spark] MaxGekk commented on a diff in pull request #42752: [SPARK-45033][SQL] Support maps by parameterized `sql()`

2023-09-03 Thread via GitHub

MaxGekk commented on code in PR #42752: URL: https://github.com/apache/spark/pull/42752#discussion_r1314453024 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/parameters.scala: ## @@ -96,7 +96,11 @@ case class PosParameterizedQuery(child: LogicalPlan, args:

[GitHub] [spark] MaxGekk commented on a diff in pull request #42752: [SPARK-45033][SQL] Support maps by parameterized `sql()`

2023-09-03 Thread via GitHub

MaxGekk commented on code in PR #42752: URL: https://github.com/apache/spark/pull/42752#discussion_r1314446198 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/parameters.scala: ## @@ -96,7 +96,11 @@ case class PosParameterizedQuery(child: LogicalPlan, args:

[GitHub] [spark] MaxGekk commented on a diff in pull request #42752: [SPARK-45033][SQL] Support maps by parameterized `sql()`

2023-09-03 Thread via GitHub

MaxGekk commented on code in PR #42752: URL: https://github.com/apache/spark/pull/42752#discussion_r131739 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/parameters.scala: ## @@ -96,7 +96,11 @@ case class PosParameterizedQuery(child: LogicalPlan, args: