cloud-fan commented on a change in pull request #28572:
URL: https://github.com/apache/spark/pull/28572#discussion_r427057763
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
##########
@@ -1735,8 +1735,15 @@ case class AnsiCast(child: Expression, dataType:
DataType, timeZoneId: Option[St
/**
* Cast the child expression to the target data type, but will throw error if
the cast might
* truncate, e.g. long -> int, timestamp -> data.
+ *
+ * Note UpCast will be eliminated if the child's dataType is already
DecimalType.
Review comment:
`... and target is DecimalType`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
