Github user MaxGekk commented on the issue: https://github.com/apache/spark/pull/22429 > Can we change `maxFields: Option[Int]` to `maxFields: Int` so that the caller will never forget to pass it in and it should either get it from some place or use the default value. @zsxwing Sure, but only replacing `Option[Int]` by `Int` cannot guarantee that. Need to remove default value to force a caller to pass a value to `simpleString()`
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org