Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20010#discussion_r160115072
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercionSuite.scala
---
@@ -462,27 +507,139 @@ class TypeCoercionSuite extends AnalysisTest {
ArrayType(DoubleType, containsNull = false),
Some(ArrayType(DoubleType, containsNull = true)))
widenTestWithStringPromotion(
- ArrayType(TimestampType, containsNull = false),
- ArrayType(StringType, containsNull = true),
+ ArrayType(ArrayType(IntegerType), containsNull = true),
+ ArrayType(ArrayType(LongType), containsNull = false),
+ Some(ArrayType(ArrayType(LongType), containsNull = true)))
--- End diff --
I think this is more coherent with the behavior in other parts and this is
the behavior I would expect. But I think that we should follow @gatorsmile's
suggestion and check Hive's behavior first.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
