Github user MaxGekk commented on the issue:

    https://github.com/apache/spark/pull/20849
  
    > Shall we expose encoding and add an alias for charset?
    
    It works for me too. 
    
    > Is this flexible option also a part of your public release?
    
    No, it is not. Only `charset` was exposed.
    
    As a summary, let's merge your PR 
https://github.com/apache/spark/pull/20877 . I will prepare a PR on top of your 
changes, remove flexible format of lineSep + force users to set line separator 
if charset is specified + `encoding` and `charset` as an alias. Flexible format 
of lineSep for text, csv and json will come as a separate PR. I hope it works 
for you. 



---

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

Reply via email to