Github user maropu commented on the issue:

    https://github.com/apache/spark/pull/20166
  
    BTW, the current `Datset.showString` prints rows thru `RowEncoder` 
deserializers like;
    ```
    scala> Seq(Seq(Seq(1, 2), Seq(3), Seq(4, 5, 6))).toDF("a").show(false)
    +------------------------------------------------------------+
    |a                                                           |
    +------------------------------------------------------------+
    |[WrappedArray(1, 2), WrappedArray(3), WrappedArray(4, 5, 6)]|
    +------------------------------------------------------------+
    ```
    If [we  cast them before 
prints](https://github.com/apache/spark/compare/master...maropu:CastToStringInShowString),
 we could get more simpler forms like;
    ```
    scala> Seq(Seq(Seq(1, 2), Seq(3), Seq(4, 5, 6))).toDF("a").show(false)
    +------------------------+
    |a                       |
    +------------------------+
    |[[1, 2], [3], [4, 5, 6]]|
    +------------------------+
    ```
    I'm not sure through, is this acceptable? (Probably, we might need to add a 
option to keep the old behaviour)


---

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

Reply via email to