-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-19444][ML][Documentation] Fix imports not being present in documentation #16789
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
|
@anshbansal before opening any more PRs or JIRAs please read http://spark.apache.org/contributing.html |
|
Ping @anshbansal |
|
@srowen Will read them and get back |
|
Test build #3557 has finished for PR 16789 at commit
|
|
Merged to master/2.1 |
…umentation ## What changes were proposed in this pull request? SPARK-19444 imports not being present in documentation ## How was this patch tested? Manual ## Disclaimer Contribution is original work and I license the work to the project under the project’s open source license Author: Aseem Bansal <[email protected]> Closes #16789 from anshbansal/patch-1. (cherry picked from commit aee2bd2) Signed-off-by: Sean Owen <[email protected]>
| import static org.apache.spark.sql.functions.callUDF; | ||
| import static org.apache.spark.sql.functions.col; | ||
| // $example off:untyped_ops$ | ||
| // $example off |
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.
It misses $
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.
PS I actually fixed this in my big "Java 8" PR for SPARK-19550 because I needed to get the examples to compile. If we don't get that merged soon yes let's fix this in a quick PR
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 hit this issue when I tried to review a Documentation-related PR. Thus, I submitted the PR.
…amount is not equal to end indices amount ### What changes were proposed in this pull request? ``` Liquid Exception: Start indices amount is not equal to end indices amount, see /Users/xiao/IdeaProjects/sparkDelivery/docs/../examples/src/main/java/org/apache/spark/examples/ml/JavaTokenizerExample.java. in ml-features.md ``` So far, the build is broken after merging #16789 This PR is to fix it. ## How was this patch tested? Manual Author: Xiao Li <[email protected]> Closes #16908 from gatorsmile/docMLFix. (cherry picked from commit 855a1b7) Signed-off-by: Sean Owen <[email protected]>
…amount is not equal to end indices amount ### What changes were proposed in this pull request? ``` Liquid Exception: Start indices amount is not equal to end indices amount, see /Users/xiao/IdeaProjects/sparkDelivery/docs/../examples/src/main/java/org/apache/spark/examples/ml/JavaTokenizerExample.java. in ml-features.md ``` So far, the build is broken after merging #16789 This PR is to fix it. ## How was this patch tested? Manual Author: Xiao Li <[email protected]> Closes #16908 from gatorsmile/docMLFix.
…umentation ## What changes were proposed in this pull request? SPARK-19444 imports not being present in documentation ## How was this patch tested? Manual ## Disclaimer Contribution is original work and I license the work to the project under the project’s open source license Author: Aseem Bansal <[email protected]> Closes apache#16789 from anshbansal/patch-1.
…amount is not equal to end indices amount ### What changes were proposed in this pull request? ``` Liquid Exception: Start indices amount is not equal to end indices amount, see /Users/xiao/IdeaProjects/sparkDelivery/docs/../examples/src/main/java/org/apache/spark/examples/ml/JavaTokenizerExample.java. in ml-features.md ``` So far, the build is broken after merging apache#16789 This PR is to fix it. ## How was this patch tested? Manual Author: Xiao Li <[email protected]> Closes apache#16908 from gatorsmile/docMLFix.
What changes were proposed in this pull request?
SPARK-19444 imports not being present in documentation
How was this patch tested?
Manual
Disclaimer
Contribution is original work and I license the work to the project under the project’s open source license