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

Reply via email to