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

    https://github.com/apache/spark/pull/22264#discussion_r214069522
  
    --- Diff: sql/core/src/test/scala/org/apache/spark/sql/QueryTest.scala ---
    @@ -290,6 +290,16 @@ object QueryTest {
         Row.fromSeq(row.toSeq.map {
           case null => null
           case d: java.math.BigDecimal => BigDecimal(d)
    +      // Equality of WrappedArray differs for AnyVal and AnyRef in Scala 
2.12.2+
    +      case seq: Seq[_] => seq.map {
    --- End diff --
    
    @maropu 
    
    
    ```
    scala> 1.toShort == 1.toDouble
    res9: Boolean = true
    
    scala> 1.toShort == true
    <console>:12: warning: comparing values of types Short and Boolean using 
`==' will always yield false
           1.toShort == true
                     ^
    res10: Boolean = false
    ```


---

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

Reply via email to