[SPARK-8925] [MLlib] Add @since tags to mllib.util

[sthota2014]

Went thru the history of changes the file MLUtils.scala and picked up the version that the change went in.

[mengxr]

@sthota2014 You don't need to tag private or package private method. We only need @since on public methods.

[sthota2014]

@mengxr Removed the @SInCE from the code for non-public methods.

[mengxr]

ok to test

[SparkQA]

Test build #37597 has finished for PR 7436 at commit 201e0fd.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • * @deprecated("use method without multiclass argument, which no longer has effect", "1.1.0")
  • * @deprecated("use method without multiclass argument, which no longer has effect", "1.1.0")
  • * @deprecated("use method without multiclass argument, which no longer has effect", "1.1.0")

[sthota2014]

@mengxr Any thoughts on this?

[mengxr]

I think we need to leave the @deprecated outside the JavaDoc as a Scala annotation to make it work in Scala.

[SparkQA]

Test build #38273 has finished for PR 7436 at commit 902d49b.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • * @deprecated("use method without multiclass argument, which no longer has effect", "1.1.0")
  • * @deprecated("use method without multiclass argument, which no longer has effect", "1.1.0")

[sthota2014]

@mengxr Any thoughts on this ?

[sthota2014]

@mengxr can you please help me complete the cycle.

[mengxr]

We still need to keep this as an annotation. @deprecated in comments is a JavaDoc tag, while @deprecated outside comments is a Scala annotation, which helps generate compiler warnings.

We should keep each PR minimal, so please keep all @deprecated annotation unchanged.

[SparkQA]

Test build #39905 has finished for PR 7436 at commit 865149c.

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

[sthota2014]

@mengxr Any thoughts on this ?

[mengxr]

LGTM. Merged into master and branch-1.5. Thanks!

[rxin]

How come the Jenkins run here didn't trigger test failures?