Skip to content

Conversation

@JBauerKogentix
Copy link
Contributor

MultilayerPerceptronClassifier had 4 occurrences

What changes were proposed in this pull request?

(Please fill in changes proposed in this fix)

How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Please review http://spark.apache.org/contributing.html before opening a pull request.

MultilayerPerceptronClassifier had 4 occurrences
@jkbradley
Copy link
Member

ok to test

@jkbradley
Copy link
Member

LGTM pending tests
Thanks for finding & fixing this!

Would you mind creating a JIRA and linking it to https://issues.apache.org/jira/browse/SPARK-21856 ? I'd like a tracking JIRA since I'd like to backport it to branch-2.3.

@SparkQA
Copy link

SparkQA commented Apr 10, 2018

Test build #89153 has finished for PR 21030 at commit dd15d44.

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

@jkbradley
Copy link
Member

Actually, forget the JIRA; I'll just merge it with master and branch-2.3 as is.

asfgit pushed a commit that referenced this pull request Apr 11, 2018
MultilayerPerceptronClassifier had 4 occurrences

## What changes were proposed in this pull request?

(Please fill in changes proposed in this fix)

## How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Please review http://spark.apache.org/contributing.html before opening a pull request.

Author: JBauerKogentix <[email protected]>

Closes #21030 from JBauerKogentix/patch-1.

(cherry picked from commit 9d960de)
Signed-off-by: Joseph K. Bradley <[email protected]>
@asfgit asfgit closed this in 9d960de Apr 11, 2018
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.

3 participants