Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-21 Thread via GitHub
MaxGekk commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1461103218 ## sql/core/src/test/scala/org/apache/spark/sql/errors/QueryCompilationErrorsSuite.scala: ## @@ -17,17 +17,17 @@ package org.apache.spark.sql.errors -import

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-21 Thread via GitHub
MaxGekk commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1461103077 ## sql/core/src/test/scala/org/apache/spark/sql/errors/QueryCompilationErrorsSuite.scala: ## @@ -17,17 +17,17 @@ package org.apache.spark.sql.errors -import

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880019229 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195529243。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880018760 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195528265。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880018252 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195526981。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880017451 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195525426。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880016835 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195524488。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
llkj1 commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880016324 感谢您与我们联系。(Thank you for contacting us.) 我们收到了您的请求,并会在一至两个工作日内给您回复。您的案例编号是 102195523219。 想了解更多与开发相关主题的信息,请访问

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2024-01-07 Thread via GitHub
MaxGekk commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1880015640 @hannahkamundson Please, check the failed tests. It seems they are related to your changes: ``` Map("expression" -> ""coalesce(1, a, a)"", "inputTypes" -> "["INT", "STRING",

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-25 Thread via GitHub
MaxGekk commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1436132674 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala: ## @@ -1300,14 +1301,16 @@ trait ComplexTypeMergingExpression extends

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-20 Thread via GitHub
hannahkamundson commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1433134058 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala: ## @@ -1300,14 +1301,16 @@ trait ComplexTypeMergingExpression

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-20 Thread via GitHub
hannahkamundson commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1433134058 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala: ## @@ -1300,14 +1301,16 @@ trait ComplexTypeMergingExpression

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-20 Thread via GitHub
hannahkamundson commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1433131387 ## common/utils/src/main/resources/error/error-classes.json: ## @@ -500,6 +500,24 @@ ], "sqlState" : "22004" }, +

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-20 Thread via GitHub
MaxGekk commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1433115102 ## common/utils/src/main/resources/error/error-classes.json: ## @@ -500,6 +500,24 @@ ], "sqlState" : "22004" }, + "COMPLEX_EXPRESSION_UNSUPPORTED_INPUT"

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-20 Thread via GitHub
MaxGekk commented on code in PR #44336: URL: https://github.com/apache/spark/pull/44336#discussion_r1433111027 ## common/utils/src/main/scala/org/apache/spark/SparkException.scala: ## @@ -106,6 +106,21 @@ object SparkException { messageParameters = Map("message" -> msg),

Re: [PR] [SPARK-42332][SQL][DOCS] Changing the require to a SparkException in ComplexTypeMergingExpression [spark]

2023-12-19 Thread via GitHub
MaxGekk commented on PR #44336: URL: https://github.com/apache/spark/pull/44336#issuecomment-1863956845 @hannahkamundson Could you remove the unused imports, see the failed GitHub actions: ``` [error]