Github user mateiz commented on the pull request:

    https://github.com/apache/spark/pull/2274#issuecomment-54564083
  
    Just a nit, it should probably be called 
repartitionAndSortWithinPartition*s*.
    
    Also, this name is pretty long. Another one I'd reconsider is 
`repartitionWithSort`, but not sure what other people think.
    
    Finally I think it should be a policy to add all these APIs to Python, and 
implement them there too. Basically there are two options -- if you're doing 
this to support a slightly easier transition from MR jobs, but you don't want 
to do it in Python, you could just have it as a document, or an example, or 
maybe even a third-party package that takes a Hadoop JobConf and runs it on 
Spark. But if you want it in Spark, we need to put it in each language. The 
reason is to allow people to easily read code in one supported language and run 
it in others -- it's always disappointing when some operators turn out to be 
missing in yours.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to