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
[SPARK-15134][EXAMPLE] Indent SparkSession builder patterns and update binary_classification_metrics_example.py #12911
Conversation
…ry_classification_metrics_example.py
Hi, @andrewor14 . |
streamId, | ||
msg.blockIds.length, | ||
client.getClientId(), | ||
NettyUtils.getRemoteAddress(client.getChannel())); |
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.
ok, but in the future I would keep unrelated changes separate
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.
Oh, I see. I'll keep it in mind.
Test build #57809 has finished for PR 12911 at commit
|
retest this please |
Test build #57826 has finished for PR 12911 at commit
|
Merging into master 2.0. Thanks! |
…e binary_classification_metrics_example.py ## What changes were proposed in this pull request? This issue addresses the comments in SPARK-15031 and also fix java-linter errors. - Use multiline format in SparkSession builder patterns. - Update `binary_classification_metrics_example.py` to use `SparkSession`. - Fix Java Linter errors (in SPARK-13745, SPARK-15031, and so far) ## How was this patch tested? After passing the Jenkins tests and run `dev/lint-java` manually. Author: Dongjoon Hyun <dongjoon@apache.org> Closes #12911 from dongjoon-hyun/SPARK-15134. (cherry picked from commit 2c170dd) Signed-off-by: Andrew Or <andrew@databricks.com>
Thank you, @andrewor14 ! |
What changes were proposed in this pull request?
This issue addresses the comments in SPARK-15031 and also fix java-linter errors.
binary_classification_metrics_example.py
to useSparkSession
.How was this patch tested?
After passing the Jenkins tests and run
dev/lint-java
manually.