From facaf1cda0fae41c86a860265b562f38acd63605 Mon Sep 17 00:00:00 2001
From: Kazuaki Ishizaki <ishizaki@jp.ibm.com>
Date: Wed, 6 Dec 2017 04:23:45 +0000
Subject: [PATCH] address review comments

---
 .../apache/spark/sql/catalyst/expressions/arithmetic.scala  | 6 +++---
 1 file changed, 3 insertions(+), 3 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 7a6d817c07544..c2b07428d3f05 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
@@ -602,7 +602,7 @@ case class Least(children: Seq[Expression]) extends Expression {
 
   override def doGenCode(ctx: CodegenContext, ev: ExprCode): ExprCode = {
     val evalChildren = children.map(_.genCode(ctx))
-    val isNull = ctx.freshName("isNull")
+    val isNull = ctx.freshName("leastTmpIsNull")
     ctx.addMutableState(ctx.JAVA_BOOLEAN, isNull)
     val evals = evalChildren.map(eval =>
       s"""
@@ -631,7 +631,7 @@ case class Least(children: Seq[Expression]) extends Expression {
       $isNull = true;
       ${ctx.javaType(dataType)} ${ev.value} = ${ctx.defaultValue(dataType)};
       $codes
-      boolean ${ev.isNull} = $isNull;""")
+      final boolean ${ev.isNull} = $isNull;""")
   }
 }
 
@@ -710,6 +710,6 @@ case class Greatest(children: Seq[Expression]) extends Expression {
       $isNull = true;
       ${ctx.javaType(dataType)} ${ev.value} = ${ctx.defaultValue(dataType)};
       $codes
-      boolean ${ev.isNull} = $isNull;""")
+      final boolean ${ev.isNull} = $isNull;""")
   }
 }