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

Posted to reviews@spark.apache.org by cloud-fan <gi...@git.apache.org> on 2018/10/11 02:50:21 UTC

[GitHub] spark pull request #22309: [SPARK-20384][SQL] Support value class in schema ...

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

    https://github.com/apache/spark/pull/22309#discussion_r224300113
  
    --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/ScalaReflectionSuite.scala ---
    @@ -108,6 +108,16 @@ object TestingUDT {
       }
     }
     
    +object TestingValueClass {
    +  case class IntWrapper(i: Int) extends AnyVal
    --- End diff --
    
    does value class must be a case class?


---

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