Addres comments

HyukjinKwon · HyukjinKwon · commit 05dc91695139 · 2020-03-25T09:19:34.000+09:00
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
@@ -272,7 +272,7 @@ abstract class CastBase extends UnaryExpression with TimeZoneAwareExpression wit
     }
   }
 
-  override def nullable: Boolean = Cast.forceNullable(child.dataType, dataType) || child.nullable
+  override def nullable: Boolean = child.nullable || Cast.forceNullable(child.dataType, dataType)
 
   protected def ansiEnabled: Boolean
 

Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,7 @@ abstract class CastBase extends UnaryExpression with TimeZoneAwareExpression wit`
`272`	`272`	`}`
`273`	`273`	`}`
`274`	`274`
`275`		`- override def nullable: Boolean = Cast.forceNullable(child.dataType, dataType) \|\| child.nullable`
	`275`	`+ override def nullable: Boolean = child.nullable \|\| Cast.forceNullable(child.dataType, dataType)`
`276`	`276`
`277`	`277`	`protected def ansiEnabled: Boolean`
`278`	`278`