{"action":"created","issue":{"url":"https://api.github.com/repos/apache/mahout/issues/76","labels_url":"https://api.github.com/repos/apache/mahout/issues/76/labels{/name}","comments_url":"https://api.github.com/repos/apache/mahout/issues/76/comments","events_url":"https://api.github.com/repos/apache/mahout/issues/76/events","html_url":"https://github.com/apache/mahout/pull/76","id":58116812,"number":76,"title":"General cleanup of Scala Similarity and Drivers","user":{"login":"pferrel","id":326731,"avatar_url":"https://avatars.githubusercontent.com/u/326731?v=3","gravatar_id":"","url":"https://api.github.com/users/pferrel","html_url":"https://github.com/pferrel","followers_url":"https://api.github.com/users/pferrel/followers","following_url":"https://api.github.com/users/pferrel/following{/other_user}","gists_url":"https://api.github.com/users/pferrel/gists{/gist_id}","starred_url":"https://api.github.com/users/pferrel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pferrel/subscriptions","organizations_url":"https://api.github.com/users/pferrel/orgs","repos_url":"https://api.github.com/users/pferrel/repos","events_url":"https://api.github.com/users/pferrel/events{/privacy}","received_events_url":"https://api.github.com/users/pferrel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":20,"created_at":"2015-02-18T19:39:37Z","updated_at":"2015-03-02T18:23:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/mahout/pulls/76","html_url":"https://github.com/apache/mahout/pull/76","diff_url":"https://github.com/apache/mahout/pull/76.diff","patch_url":"https://github.com/apache/mahout/pull/76.patch"},"body":"Many small cleanup changes:\r\n\r\n * simplified drivers\r\n * removed any reference to cross-indicator and most references to indicator\r\n * shortened long lines\r\n * cleaned up comments and scaladoc annotations\r\n * replaced old use of o.a.m.math.Pair with Scala tuples\r\n\r\nTested ```spark-itemsimilarity``` on cluster but not naive Bayes stuff\r\n\r\nDecided not to remove scopt. Removing it would be more trouble than it's worth given how small the lib is. It still may be worth using case classes for options to get rid of verbose casts but doesn't seem pressing.\r\n\r\nThis PR doesn't touch the job.jar assembly in the spark module. I have a pare down of that waiting for other refactoring  commits from @dlyubimov.\r\n\r\nNo other work planned on this PR"},"comment":{"url":"https://api.github.com/repos/apache/mahout/issues/comments/76769257","html_url":"https://github.com/apache/mahout/pull/76#issuecomment-76769257","issue_url":"https://api.github.com/repos/apache/mahout/issues/76","id":76769257,"user":{"login":"pferrel","id":326731,"avatar_url":"https://avatars.githubusercontent.com/u/326731?v=3","gravatar_id":"","url":"https://api.github.com/users/pferrel","html_url":"https://github.com/pferrel","followers_url":"https://api.github.com/users/pferrel/followers","following_url":"https://api.github.com/users/pferrel/following{/other_user}","gists_url":"https://api.github.com/users/pferrel/gists{/gist_id}","starred_url":"https://api.github.com/users/pferrel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pferrel/subscriptions","organizations_url":"https://api.github.com/users/pferrel/orgs","repos_url":"https://api.github.com/users/pferrel/repos","events_url":"https://api.github.com/users/pferrel/events{/privacy}","received_events_url":"https://api.github.com/users/pferrel/received_events","type":"User","site_admin":false},"created_at":"2015-03-02T18:23:16Z","updated_at":"2015-03-02T18:23:16Z","body":"Seems like we should avoid 1.2 and 1.2.1 though we could tell someone how to make it work and only item and rowsimilairty are broken anyway.\r\n\r\nSo I'm testing this with Spark 1.1.1 and will push that for now. All that is required to upgrade to 1.2 or 1.2.1 would be the root pom Spark version change and build from source"}}