{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/9513","labels_url":"https://api.github.com/repos/apache/spark/issues/9513/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/9513/comments","events_url":"https://api.github.com/repos/apache/spark/issues/9513/events","html_url":"https://github.com/apache/spark/pull/9513","id":115414811,"number":9513,"title":"[SPARK-5565] [ML] LDA wrapper for Pipelines API","user":{"login":"jkbradley","id":5084283,"avatar_url":"https://avatars.githubusercontent.com/u/5084283?v=3","gravatar_id":"","url":"https://api.github.com/users/jkbradley","html_url":"https://github.com/jkbradley","followers_url":"https://api.github.com/users/jkbradley/followers","following_url":"https://api.github.com/users/jkbradley/following{/other_user}","gists_url":"https://api.github.com/users/jkbradley/gists{/gist_id}","starred_url":"https://api.github.com/users/jkbradley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jkbradley/subscriptions","organizations_url":"https://api.github.com/users/jkbradley/orgs","repos_url":"https://api.github.com/users/jkbradley/repos","events_url":"https://api.github.com/users/jkbradley/events{/privacy}","received_events_url":"https://api.github.com/users/jkbradley/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":20,"created_at":"2015-11-06T01:26:26Z","updated_at":"2015-11-10T10:09:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/9513","html_url":"https://github.com/apache/spark/pull/9513","diff_url":"https://github.com/apache/spark/pull/9513.diff","patch_url":"https://github.com/apache/spark/pull/9513.patch"},"body":"This adds LDA to spark.ml, the Pipelines API.  It follows the design doc in the JIRA: [https://issues.apache.org/jira/browse/SPARK-5565], with one major change:\r\n* I eliminated doc IDs.  These are not necessary with DataFrames since the user can add an ID column as needed.\r\n\r\nNote: This will conflict with [https://github.com/apache/spark/pull/9484], but I'll try to merge [https://github.com/apache/spark/pull/9484] first and then rebase this PR.\r\n\r\nCC: @hhbyyh @feynmanliang  If you have a chance to make a pass, that'd be really helpful--thanks!  Now that I'm done traveling & this PR is almost ready, I'll see about reviewing other PRs critical for 1.6.\r\n\r\nCC: @mengxr "},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/155376552","html_url":"https://github.com/apache/spark/pull/9513#issuecomment-155376552","issue_url":"https://api.github.com/repos/apache/spark/issues/9513","id":155376552,"user":{"login":"feynmanliang","id":990069,"avatar_url":"https://avatars.githubusercontent.com/u/990069?v=3","gravatar_id":"","url":"https://api.github.com/users/feynmanliang","html_url":"https://github.com/feynmanliang","followers_url":"https://api.github.com/users/feynmanliang/followers","following_url":"https://api.github.com/users/feynmanliang/following{/other_user}","gists_url":"https://api.github.com/users/feynmanliang/gists{/gist_id}","starred_url":"https://api.github.com/users/feynmanliang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/feynmanliang/subscriptions","organizations_url":"https://api.github.com/users/feynmanliang/orgs","repos_url":"https://api.github.com/users/feynmanliang/repos","events_url":"https://api.github.com/users/feynmanliang/events{/privacy}","received_events_url":"https://api.github.com/users/feynmanliang/received_events","type":"User","site_admin":false},"created_at":"2015-11-10T10:09:28Z","updated_at":"2015-11-10T10:09:28Z","body":"+1 on the renames\n\nOn Tue, Nov 10, 2015, 02:48 Apache Spark QA <notifications@github.com>\nwrote:\n\n> *Test build #2025 has finished\n> <https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2025/consoleFull>*\n> for PR 9513 at commit 16a061c\n> <https://github.com/apache/spark/commit/16a061ca4df6abb59b1cff6695debac7492260ab>\n> .\n>\n>    - This patch passes all tests.\n>    - This patch merges cleanly.\n>    - This patch adds the following public classes *(experimental)*:\\n * class\n>    LDA @Since(\\\"1.6.0\\\") (\\n\n>\n> —\n>\n>\n> Reply to this email directly or view it on GitHub\n> <https://github.com/apache/spark/pull/9513#issuecomment-155266291>.\n>\n"}}