{"action":"created","issue":{"url":"https://api.github.com/repos/jigold/hail/issues/8","repository_url":"https://api.github.com/repos/jigold/hail","labels_url":"https://api.github.com/repos/jigold/hail/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/jigold/hail/issues/8/comments","events_url":"https://api.github.com/repos/jigold/hail/issues/8/events","html_url":"https://github.com/jigold/hail/pull/8","id":279237691,"number":8,"title":"Created RVD","user":{"login":"cseed","id":1244990,"avatar_url":"https://avatars3.githubusercontent.com/u/1244990?v=4","gravatar_id":"","url":"https://api.github.com/users/cseed","html_url":"https://github.com/cseed","followers_url":"https://api.github.com/users/cseed/followers","following_url":"https://api.github.com/users/cseed/following{/other_user}","gists_url":"https://api.github.com/users/cseed/gists{/gist_id}","starred_url":"https://api.github.com/users/cseed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cseed/subscriptions","organizations_url":"https://api.github.com/users/cseed/orgs","repos_url":"https://api.github.com/users/cseed/repos","events_url":"https://api.github.com/users/cseed/events{/privacy}","received_events_url":"https://api.github.com/users/cseed/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-12-05T03:56:52Z","updated_at":"2017-12-05T10:35:02Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/jigold/hail/pulls/8","html_url":"https://github.com/jigold/hail/pull/8","diff_url":"https://github.com/jigold/hail/pull/8.diff","patch_url":"https://github.com/jigold/hail/pull/8.patch"},"body":"And related classes:\r\n\r\nOrderedRDD2 => OrderedRVD\r\nOrderedRDD2Type => OrderedRVType\r\nOrderedrDD2Partitioner => OrderedRVPartitioner\r\nPartitionKeyInfo2 => OrderedRVPartitionInfo\r\n\r\nI moved those classes into is.hail.rvd.  RVD and OrderedRVD don't extend RDD anymore.  All map/mapPartitions functions take a suitable type.\r\n\r\nSurprisingly few changes outside OrderedRDD/RVD.  98% of this is moving things around.  (Sorry for lack of good diffs, I realize in retrospect I should have made the changes in place and kept the moves for a separate PR.)\r\n\r\nI slightly strengthened the invariant of persist: persisting something already persisted at a new storage level throws an exception.  I think everything else behaves the same.\r\n\r\n@tpoterba FYI"},"comment":{"url":"https://api.github.com/repos/jigold/hail/issues/comments/349263895","html_url":"https://github.com/jigold/hail/pull/8#issuecomment-349263895","issue_url":"https://api.github.com/repos/jigold/hail/issues/8","id":349263895,"user":{"login":"tpoterba","id":10562794,"avatar_url":"https://avatars2.githubusercontent.com/u/10562794?v=4","gravatar_id":"","url":"https://api.github.com/users/tpoterba","html_url":"https://github.com/tpoterba","followers_url":"https://api.github.com/users/tpoterba/followers","following_url":"https://api.github.com/users/tpoterba/following{/other_user}","gists_url":"https://api.github.com/users/tpoterba/gists{/gist_id}","starred_url":"https://api.github.com/users/tpoterba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tpoterba/subscriptions","organizations_url":"https://api.github.com/users/tpoterba/orgs","repos_url":"https://api.github.com/users/tpoterba/repos","events_url":"https://api.github.com/users/tpoterba/events{/privacy}","received_events_url":"https://api.github.com/users/tpoterba/received_events","type":"User","site_admin":false},"created_at":"2017-12-05T10:35:02Z","updated_at":"2017-12-05T10:35:02Z","author_association":"NONE","body":"I'm not sure I totally like persist erroring. In particular, there are places where we should definitely persist key tables computed from VSMs automatically without the user needing to do so, and so this could cause a user to encounter a persist error without calling it twice themselves."}}
{"actor":{"display_login":"tpoterba"}}