Uh oh!
There was an error while loading. Please reload this page.
[MINOR] Fix Typos - #13078
Conversation
SparkQA
commented
May 12, 2016
Test build #58488 has started for PR 13078 at commit |
shaneknapp
commented
May 12, 2016
i will retrigger this build once maintenance is over. |
There was a problem hiding this comment.
Y'know, while we're at it, the matrix args in this whole class don't match the docs. The arg here and similarly elsewhere can be called "A" not "a".
Also this should check that A.rows == y.length for consistency.
There was a problem hiding this comment.
Right. I will rename the matrix args to upper and add this missing checking.
shaneknapp
commented
May 12, 2016
jenkins, test this please |
1 similar comment
shaneknapp
commented
May 12, 2016
jenkins, test this please |
SparkQA
commented
May 12, 2016
Test build #58497 has finished for PR 13078 at commit
|
SparkQA
commented
May 13, 2016
Test build #58537 has finished for PR 13078 at commit
|
rxin
commented
May 20, 2016
@srowen we should backport the doc fixes into branch-2.0. |
What changes were proposed in this pull request?
1,Rename matrix args in BreezeUtil to upper to match the doc
2,Fix several typos in ML and SQL
How was this patch tested?
manual tests