This is an automated email from the ASF dual-hosted git repository.
maxgekk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 04339e30dbd [SPARK-44975][SQL] Remove BinaryArithmetic useless
override resolved
04339e30dbd is described below
commit 04339e30dbdda2805edbac7e1e3cd8dfb5c3c608
Author: Jia Fan <[email protected]>
AuthorDate: Sat Aug 26 21:11:20 2023 +0300
[SPARK-44975][SQL] Remove BinaryArithmetic useless override resolved
### What changes were proposed in this pull request?
Remove `BinaryArithmetic` useless override resolved, it is exactly the same
as the abstract class `Expression`
### Why are the changes needed?
remove useless logic
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
exist test
### Was this patch authored or co-authored using generative AI tooling?
Closes #42689 from Hisoka-X/SPARK-44975_remove_resolved_override.
Authored-by: Jia Fan <[email protected]>
Signed-off-by: Max Gekk <[email protected]>
---
.../scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala | 2 --
1 file changed, 2 deletions(-)
diff --git
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
index 31d4d71cd40..2d9bccc0854 100644
---
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
+++
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
@@ -264,8 +264,6 @@ abstract class BinaryArithmetic extends BinaryOperator
final override val nodePatterns: Seq[TreePattern] = Seq(BINARY_ARITHMETIC)
- override lazy val resolved: Boolean = childrenResolved &&
checkInputDataTypes().isSuccess
-
override def initQueryContext(): Option[SQLQueryContext] = {
if (failOnError) {
Some(origin.context)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
