Skip to content

Conversation

@yanboliang
Copy link
Contributor

Check and add miss docs for PySpark ML (this issue only check miss docs for o.a.s.ml not o.a.s.mllib).

@SparkQA
Copy link

SparkQA commented Aug 9, 2015

Test build #40265 has finished for PR 8059 at commit de59f54.

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about starting with a short version: "It supports both Multinomial and Binomial NB."
And then continuing with details: "Multinomial NB (link) can handle..."

@jkbradley
Copy link
Member

Just those 2 small comments.

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #40626 has finished for PR 8059 at commit 85770d1.

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

@jkbradley
Copy link
Member

LGTM, thanks! Merging with master and branch-1.5

asfgit pushed a commit that referenced this pull request Aug 12, 2015
Check and add miss docs for PySpark ML (this issue only check miss docs for o.a.s.ml not o.a.s.mllib).

Author: Yanbo Liang <ybliang8@gmail.com>

Closes #8059 from yanboliang/SPARK-9766.

(cherry picked from commit 762bacc)
Signed-off-by: Joseph K. Bradley <joseph@databricks.com>
@asfgit asfgit closed this in 762bacc Aug 12, 2015
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
Check and add miss docs for PySpark ML (this issue only check miss docs for o.a.s.ml not o.a.s.mllib).

Author: Yanbo Liang <ybliang8@gmail.com>

Closes apache#8059 from yanboliang/SPARK-9766.
@yanboliang yanboliang deleted the SPARK-9766 branch August 26, 2015 07:08
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