{"action":"created","issue":{"url":"https://api.github.com/repos/apache/mahout/issues/133","labels_url":"https://api.github.com/repos/apache/mahout/issues/133/labels{/name}","comments_url":"https://api.github.com/repos/apache/mahout/issues/133/comments","events_url":"https://api.github.com/repos/apache/mahout/issues/133/events","html_url":"https://github.com/apache/mahout/pull/133","id":82535774,"number":133,"title":"MAHOUT-1706-0.10.x","user":{"login":"andrewpalumbo","id":7681565,"avatar_url":"https://avatars.githubusercontent.com/u/7681565?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewpalumbo","html_url":"https://github.com/andrewpalumbo","followers_url":"https://api.github.com/users/andrewpalumbo/followers","following_url":"https://api.github.com/users/andrewpalumbo/following{/other_user}","gists_url":"https://api.github.com/users/andrewpalumbo/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewpalumbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewpalumbo/subscriptions","organizations_url":"https://api.github.com/users/andrewpalumbo/orgs","repos_url":"https://api.github.com/users/andrewpalumbo/repos","events_url":"https://api.github.com/users/andrewpalumbo/events{/privacy}","received_events_url":"https://api.github.com/users/andrewpalumbo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-29T18:30:53Z","updated_at":"2015-06-09T22:27:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/mahout/pulls/133","html_url":"https://github.com/apache/mahout/pull/133","diff_url":"https://github.com/apache/mahout/pull/133.diff","patch_url":"https://github.com/apache/mahout/pull/133.patch"},"body":"The mahout distribution currently is shipping ~56 MB of dependency jars in the /lib directory of the distribution. These are only added to the classpath by /bin/mahout in the binary distribution. It seems that we can remove them from the distribution. (we need to get the size of the distribution down).\r\n\r\nThis branch still needs testing on an cluster."},"comment":{"url":"https://api.github.com/repos/apache/mahout/issues/comments/110522411","html_url":"https://github.com/apache/mahout/pull/133#issuecomment-110522411","issue_url":"https://api.github.com/repos/apache/mahout/issues/133","id":110522411,"user":{"login":"andrewpalumbo","id":7681565,"avatar_url":"https://avatars.githubusercontent.com/u/7681565?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewpalumbo","html_url":"https://github.com/andrewpalumbo","followers_url":"https://api.github.com/users/andrewpalumbo/followers","following_url":"https://api.github.com/users/andrewpalumbo/following{/other_user}","gists_url":"https://api.github.com/users/andrewpalumbo/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewpalumbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewpalumbo/subscriptions","organizations_url":"https://api.github.com/users/andrewpalumbo/orgs","repos_url":"https://api.github.com/users/andrewpalumbo/repos","events_url":"https://api.github.com/users/andrewpalumbo/events{/privacy}","received_events_url":"https://api.github.com/users/andrewpalumbo/received_events","type":"User","site_admin":false},"created_at":"2015-06-09T22:27:06Z","updated_at":"2015-06-09T22:27:06Z","body":"It should not (negatively) affect the shell classpath, since all of the necessary dependencies are either packaged with {{mahout-spark-*.jars}} or in the classpath returned by {{${SPARK_HOME}/bin/compute-classpath.sh}}.  \r\n\r\nThis patch actually does not affect the classpath of a source build at all.  It only prevents the release process from packaging dependency jars into the {{/lib}} dir of the binary distribution artifact.\r\n\r\nHowever since it does add all of the jars in the {{/lib}} directory of the binary build for any mahout command line flavor (spark, MR, h2o), I have been hesitant to push it without more extensive testing (cluster and local).\r\n\r\nI am going to close this for now and investigate a better approach. \r\n\r\n     \r\n\r\n"}}