You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by xuanyuanking <gi...@git.apache.org> on 2018/04/23 15:59:52 UTC

[GitHub] spark pull request #20946: [SPARK-23565] [SQL] New error message for structu...

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

    https://github.com/apache/spark/pull/20946#discussion_r183447816
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/OffsetSeq.scala ---
    @@ -39,7 +39,9 @@ case class OffsetSeq(offsets: Seq[Option[Offset]], metadata: Option[OffsetSeqMet
        * cannot be serialized).
        */
       def toStreamProgress(sources: Seq[BaseStreamingSource]): StreamProgress = {
    -    assert(sources.size == offsets.size)
    +    assert(sources.size == offsets.size, s"There are [${offsets.size}] sources in the " +
    +      s"checkpoint offsets and now there are [${sources.size}] sources requested by the query.  " +
    --- End diff --
    
    nit: extra blank in the end


---

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