From d299e2bf2f6733a6267b7ce85e2b288608b17db3 Mon Sep 17 00:00:00 2001
From: Takuya UESHIN <ueshin@happy-camper.st>
Date: Mon, 25 Aug 2014 16:27:00 -0700
Subject: [PATCH] [SPARK-3204][SQL] MaxOf would be foldable if both left and
right are foldable.
Author: Takuya UESHIN <ueshin@happy-camper.st>
Closes #2116 from ueshin/issues/SPARK-3204 and squashes the following commits:
7d9b107 [Takuya UESHIN] Make MaxOf foldable if both left and right are foldable.
---
.../org/apache/spark/sql/catalyst/expressions/arithmetic.scala | 2 ++
1 file changed, 2 insertions(+)
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 8d90614e45..5f8b6ae10f 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
@@ -89,6 +89,8 @@ case class Remainder(left: Expression, right: Expression) extends BinaryArithmet
case class MaxOf(left: Expression, right: Expression) extends Expression {
type EvaluatedType = Any
+ override def foldable = left.foldable && right.foldable
+
override def nullable = left.nullable && right.nullable
override def children = left :: right :: Nil
--
GitLab