Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19454
  
    I think @srowen requested to fix it in a more performant way as well, for 
example, referring https://github.com/apache/spark/pull/16276, if I understood 
correctly and otherwise closing it.
    
    I don't feel strongly about adding this but I was thinking that we might 
have to go ahead given this API has been required multiple times without 
explicit objection IIUC and, looks consistent with Scala's [`flatten`](
     
https://github.com/scala/scala/blob/05016d9035ab9b1c866bd9f12fdd0491f1ea0cbb/src/library/scala/collection/generic/GenericTraversableTemplate.scala#L169).
 However, IMHO, it might be worthwhile _only if_ this PR gives a clean shot.
    
    I'd suggest to close this if we (you and other reviewers here) have to 
spend a lot of time. Workaround is quite easy anyway.


---

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

Reply via email to