-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-2061] Made splits deprecated in JavaRDDLike #1062
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how well this shows up in JavaDoc. Can you test it out?
We might need to add the java @deprecated, and then explain it has been deprecated since 1.0.1. (it is not 0.9.2).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rxin actually I don't think we should deprecate things inside of bug fix releases (people shouldn't get compile warnings for upgrading to a fix release). So this should be deprecated since 1.1.0.
|
@anantasty could you do the second part of this JIRA? It's in your description but it looks like you didn't do it: "We should also go through all cases where other API's (e.g. Python) call splits and we should change those to use the newer API." |
|
@pwendell I can certianly do that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be 1.1.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed this when merging.
The jira for the issue can be found at: https://issues.apache.org/jira/browse/SPARK-2061 Most of spark has used over to consistently using `partitions` instead of `splits`. We should do likewise and add a `partitions` method to JavaRDDLike and have `splits` just call that. We should also go through all cases where other API's (e.g. Python) call `splits` and we should change those to use the newer API. Author: Anant <[email protected]> Closes apache#1062 from anantasty/SPARK-2061 and squashes the following commits: b83ce6b [Anant] Fixed syntax issue 21f9210 [Anant] Fixed version number in deprecation string 9315b76 [Anant] made related changes to use partitions in python api 8c62dd1 [Anant] Made splits deprecated in JavaRDDLike
The jira for the issue can be found at: https://issues.apache.org/jira/browse/SPARK-2061 Most of spark has used over to consistently using `partitions` instead of `splits`. We should do likewise and add a `partitions` method to JavaRDDLike and have `splits` just call that. We should also go through all cases where other API's (e.g. Python) call `splits` and we should change those to use the newer API. Author: Anant <[email protected]> Closes apache#1062 from anantasty/SPARK-2061 and squashes the following commits: b83ce6b [Anant] Fixed syntax issue 21f9210 [Anant] Fixed version number in deprecation string 9315b76 [Anant] made related changes to use partitions in python api 8c62dd1 [Anant] Made splits deprecated in JavaRDDLike
- add 'proxy' authentication handler
- add 'proxy' authentication handler
The jira for the issue can be found at: https://issues.apache.org/jira/browse/SPARK-2061
Most of spark has used over to consistently using
partitionsinstead ofsplits. We should do likewise and add apartitionsmethod to JavaRDDLike and havesplitsjust call that. We should also go through all cases where other API's (e.g. Python) callsplitsand we should change those to use the newer API.