Github user MaxGekk commented on the issue:

    https://github.com/apache/spark/pull/22429
  
    @rednaxelafx Thank you for such detailed comment.
    
    > It would be better to pass the max fields argument down the call chain 
instead of changing the Spark conf.
    
    Sure, it would be better. I just afraid I will break something by changing 
signature of methods like `simpleString`
    
    > The best-performing way to do this is probably to implement a custom 
Writer
    
    I am going to replace `StringWriter` by `StringBuilderWriter`. It should be 
faster though I am not sure it matters here.
    



---

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

Reply via email to