{"action":"created","issue":{"url":"https://api.github.com/repos/amplab/tachyon/issues/729","labels_url":"https://api.github.com/repos/amplab/tachyon/issues/729/labels{/name}","comments_url":"https://api.github.com/repos/amplab/tachyon/issues/729/comments","events_url":"https://api.github.com/repos/amplab/tachyon/issues/729/events","html_url":"https://github.com/amplab/tachyon/pull/729","id":62067762,"number":729,"title":"Refactor out Under FS into separate modules (TACHYON-221)","user":{"login":"rvesse","id":2104864,"avatar_url":"https://avatars.githubusercontent.com/u/2104864?v=3","gravatar_id":"","url":"https://api.github.com/users/rvesse","html_url":"https://github.com/rvesse","followers_url":"https://api.github.com/users/rvesse/followers","following_url":"https://api.github.com/users/rvesse/following{/other_user}","gists_url":"https://api.github.com/users/rvesse/gists{/gist_id}","starred_url":"https://api.github.com/users/rvesse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rvesse/subscriptions","organizations_url":"https://api.github.com/users/rvesse/orgs","repos_url":"https://api.github.com/users/rvesse/repos","events_url":"https://api.github.com/users/rvesse/events{/privacy}","received_events_url":"https://api.github.com/users/rvesse/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-16T12:38:59Z","updated_at":"2015-03-17T15:12:41Z","closed_at":"2015-03-17T15:12:41Z","pull_request":{"url":"https://api.github.com/repos/amplab/tachyon/pulls/729","html_url":"https://github.com/amplab/tachyon/pull/729","diff_url":"https://github.com/amplab/tachyon/pull/729.diff","patch_url":"https://github.com/amplab/tachyon/pull/729.patch"},"body":"This is a resubmission of #642 and #647 and hopefully should now be sufficiently stable to be merged and pave the way for further extraction of modules based upon this pattern.\r\n\r\nIt is rebased up against the latest master (at time of submission) and as part of rebasing I have tried to strip out unnecessary formatting changes where possible.  However I haven't gone back through and stripped these out of classes that didn't have rebase conflicts so there are still some there but I hope these won't unduly hold up getting this reviewed and merged.\r\n\r\nSee the previous pull requests for general overview of the PR, main changes since last submission are as follows:\r\n\r\n- Accounts for @hsaputra's  `TachyonConf` refactoring\r\n- Adds new Under FS module JARs to the Tachyon class path in `libexec/tachyon-config.sh`\r\n- Strips out some of the unrelated formatting changes\r\n\r\nAs previously noted one advantage of this refactoring is that both local and HDFS tests are run as part of the build.  Gluster FS tests still require explicitly enabling since they require a specific environment to run.\r\n\r\nI have successfully started a Tachyon cluster and done `tachyon runTests` with both a local and HDFS under file system in my environment."},"comment":{"url":"https://api.github.com/repos/amplab/tachyon/issues/comments/82400317","html_url":"https://github.com/amplab/tachyon/pull/729#issuecomment-82400317","issue_url":"https://api.github.com/repos/amplab/tachyon/issues/729","id":82400317,"user":{"login":"rvesse","id":2104864,"avatar_url":"https://avatars.githubusercontent.com/u/2104864?v=3","gravatar_id":"","url":"https://api.github.com/users/rvesse","html_url":"https://github.com/rvesse","followers_url":"https://api.github.com/users/rvesse/followers","following_url":"https://api.github.com/users/rvesse/following{/other_user}","gists_url":"https://api.github.com/users/rvesse/gists{/gist_id}","starred_url":"https://api.github.com/users/rvesse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rvesse/subscriptions","organizations_url":"https://api.github.com/users/rvesse/orgs","repos_url":"https://api.github.com/users/rvesse/repos","events_url":"https://api.github.com/users/rvesse/events{/privacy}","received_events_url":"https://api.github.com/users/rvesse/received_events","type":"User","site_admin":false},"created_at":"2015-03-17T15:12:41Z","updated_at":"2015-03-17T15:12:41Z","body":"Closing to address formatting issues"}}