SPARK-16791_cast_struct_with_timestamp_field_fails: fix style comment…

…s from reviewers.

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala

@@ -416,7 +416,7 @@ case class Cast(child: Expression, dataType: DataType) extends UnaryExpression w
   }
 
   private[this] def cast(from: DataType, to: DataType): Any => Any = to match {
-    case dt if dt ==  from => identity[Any]
+    case dt if dt == from => identity[Any]
     case StringType => castToString(from)
     case BinaryType => castToBinary(from)
     case DateType => castToDate(from)

sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala

@@ -728,16 +728,16 @@ class CastSuite extends SparkFunSuite with ExpressionEvalHelper {
   }
 
   test("cast struct with a timestamp field") {
-    val origSchema = StructType(
-      Seq(StructField( "tsField", TimestampType, false ))
+    val originalSchema = StructType(
+      Seq(StructField( "tsField", TimestampType, nullable = false ))
     )
-    val trgSchema = StructType(
+    val targetSchema = StructType(
       // nine out of ten times I'm casting a struct, it's to normalize its fields nullability
-      Seq(StructField( "tsField", TimestampType, true ))
+      Seq(StructField( "tsField", TimestampType, nullable = true ))
     )
-    val inp = Literal.create(InternalRow(0L), origSchema)
+    val inp = Literal.create(InternalRow(0L), originalSchema)
     val expected = InternalRow(0L)
-    checkEvaluation( Cast(inp, trgSchema), expected)
+    checkEvaluation(cast(inp, targetSchema), expected )
   }
 
   test("complex casting") {