Skip to content

Conversation

@BryanCutler
Copy link
Member

What changes were proposed in this pull request?

Some appNames in ML examples are incorrect, mostly in PySpark but one in Scala. This corrects the names.

How was this patch tested?

Style, local tests

@BryanCutler
Copy link
Member Author

Not sure if this is something worth squeezing into 2.0, but I thought I'd patch it now in case.
cc @mengxr @jkbradley

@srowen
Copy link
Member

srowen commented Jun 28, 2016

LGTM

@SparkQA
Copy link

SparkQA commented Jun 28, 2016

Test build #61395 has finished for PR 13949 at commit 8f9c5cc.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

asfgit pushed a commit that referenced this pull request Jun 29, 2016
## What changes were proposed in this pull request?

Some appNames in ML examples are incorrect, mostly in PySpark but one in Scala.  This corrects the names.

## How was this patch tested?
Style, local tests

Author: Bryan Cutler <cutlerb@gmail.com>

Closes #13949 from BryanCutler/pyspark-example-appNames-fix-SPARK-16261.

(cherry picked from commit 21385d0)
Signed-off-by: Nick Pentreath <nickp@za.ibm.com>
@MLnick
Copy link
Contributor

MLnick commented Jun 29, 2016

LGTM too - I've merged to master/branch-2.0. Set the JIRA fix version to 2.0.1 for now since that seems the be what we're doing. I guess when RC2 is cut we will re-assign those ones...

@asfgit asfgit closed this in 21385d0 Jun 29, 2016
@BryanCutler
Copy link
Member Author

Thanks Nick and Sean!

@BryanCutler BryanCutler deleted the pyspark-example-appNames-fix-SPARK-16261 branch December 2, 2016 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants