Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21361#discussion_r189429002
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/complexTypesSuite.scala
 ---
    @@ -439,4 +439,17 @@ class ComplexTypesSuite extends PlanTest with 
ExpressionEvalHelper {
             .select('c as 'sCol2, 'a as 'sCol1)
         checkRule(originalQuery, correctAnswer)
       }
    +
    +  test("SPARK-24313: support complex types as map keys") {
    +    val mb0 = Literal.create(
    +      Map(Array[Byte](1, 2) -> "1", Array[Byte](3, 4) -> null, 
Array[Byte](2, 1) -> "2"),
    +      MapType(BinaryType, StringType))
    --- End diff --
    
    shall we test `ArrayType`to reflect the test name?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to