[SPARK-25150][SQL] Fix review findings 4

Change-Id: I928e9a56410c00c4fbbd11c8e91b7993a4bc1878
apache · Sep 5, 2018 · 809b8a8 · 809b8a8
1 parent d56f33c
commit 809b8a8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
@@ -754,7 +754,6 @@ class Analyzer(
    * a logical plan node's children.
    */
   object ResolveReferences extends Rule[LogicalPlan] {
-
     /**
      * Generate a new logical plan for the right child with different expression IDs
      * for all conflicting attributes.

diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/AttributeMap.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/AttributeMap.scala
@@ -23,7 +23,7 @@ package org.apache.spark.sql.catalyst.expressions
  * of the name, or the expected nullability).
  */
 object AttributeMap {
-  var empty = new AttributeMap(Map.empty)
+  val empty = new AttributeMap(Map.empty)
 
   def apply[A](kvs: Seq[(Attribute, A)]): AttributeMap[A] = {
     new AttributeMap(kvs.map(kv => (kv._1.exprId, kv)).toMap)