Github user maropu commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21643#discussion_r198339961
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/ComplexDataSuite.scala
 ---
    @@ -104,4 +104,40 @@ class ComplexDataSuite extends SparkFunSuite {
         // The copied data should not be changed externally.
         assert(copied.getStruct(0, 1).getUTF8String(0).toString == "a")
       }
    +
    +  test("SPARK-24659: GenericArrayData.equals should respect element type 
differences") {
    +    import scala.reflect.ClassTag
    --- End diff --
    
    nit:  you can move this import to the head of this file.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to