[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1040017671 ## sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala: ## @@ -637,17 +637,21 @@ private[sql] object QueryCompilationErrors extends

[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1035862190 ## sql/core/src/test/resources/sql-tests/results/ansi/string-functions.sql.out: ## @@ -848,7 +849,8 @@ org.apache.spark.sql.AnalysisException "messageParameters" :

[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1032047234 ## sql/core/src/test/resources/sql-tests/results/ansi/string-functions.sql.out: ## @@ -848,7 +849,8 @@ org.apache.spark.sql.AnalysisException "messageParameters" :

[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1030185866 ## sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala: ## @@ -637,17 +637,24 @@ private[sql] object QueryCompilationErrors extends

[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1026264084 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala: ## @@ -146,7 +147,10 @@ object FunctionRegistryBase {

[GitHub] [spark] MaxGekk commented on a diff in pull request #38664: [SPARK-41147][SQL] Assign a name to the legacy error class `_LEGACY_ERROR_TEMP_1042`

MaxGekk commented on code in PR #38664: URL: https://github.com/apache/spark/pull/38664#discussion_r1024815444 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala: ## @@ -146,7 +146,7 @@ object FunctionRegistryBase {