Skip to content
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

[MLlib][TYPO] Replace LeastSquaresAggregator with LogisticAggregator #16903

Closed
wants to merge 1 commit into from

Conversation

moutai
Copy link
Contributor

@moutai moutai commented Feb 12, 2017

What changes were proposed in this pull request?

Replace LeastSquaresAggregator with LogisticAggregator in the require statement of the merge op.

How was this patch tested?

Simple message fix.

@srowen
Copy link
Member

srowen commented Feb 17, 2017

Sure, I wonder if there are other similar typos in versions of this message that may be copied and pasted elsewhere? can you take a quick look?

@SparkQA
Copy link

SparkQA commented Feb 18, 2017

Test build #3579 has finished for PR 16903 at commit 669820c.

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

@srowen
Copy link
Member

srowen commented Feb 18, 2017

(Doesn't look like it, so merged to master)

@asfgit asfgit closed this in 21c7d3c Feb 18, 2017
@moutai
Copy link
Contributor Author

moutai commented Feb 18, 2017

Thanks,
Yes I think the rest of the messages look fine on this file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants