You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by maropu <gi...@git.apache.org> on 2018/08/30 14:06:39 UTC

[GitHub] spark pull request #22264: [SPARK-25256][SQL][TEST] Plan mismatch errors in ...

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

    https://github.com/apache/spark/pull/22264#discussion_r214044804
  
    --- 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 --
    
    Don't need `case b: java.lang.Boolean => b.booleanValue()`?


---

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