Skip to content

Conversation

@mengxr
Copy link
Contributor

@mengxr mengxr commented Apr 28, 2015

It shouldn't live directly under spark.ml.

@SparkQA
Copy link

SparkQA commented Apr 28, 2015

Test build #31151 has finished for PR 5749 at commit 53847f9.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@mengxr
Copy link
Contributor Author

mengxr commented Apr 28, 2015

Merged into master.

@asfgit asfgit closed this in f0a1f90 Apr 28, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
It shouldn't live directly under `spark.ml`.

Author: Xiangrui Meng <[email protected]>

Closes apache#5749 from mengxr/SPARK-7201 and squashes the following commits:

53847f9 [Xiangrui Meng] move Identifiable to ml.util
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
It shouldn't live directly under `spark.ml`.

Author: Xiangrui Meng <[email protected]>

Closes apache#5749 from mengxr/SPARK-7201 and squashes the following commits:

53847f9 [Xiangrui Meng] move Identifiable to ml.util
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.

2 participants