{"action":"created","issue":{"url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/61","labels_url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/61/labels{/name}","comments_url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/61/comments","events_url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/61/events","html_url":"https://github.com/edx/edx-analytics-data-api/pull/61","id":58153655,"number":61,"title":"Make API compatible with old and new answer_dist schema.","user":{"login":"dylanrhodes","id":1864903,"avatar_url":"https://avatars.githubusercontent.com/u/1864903?v=3","gravatar_id":"","url":"https://api.github.com/users/dylanrhodes","html_url":"https://github.com/dylanrhodes","followers_url":"https://api.github.com/users/dylanrhodes/followers","following_url":"https://api.github.com/users/dylanrhodes/following{/other_user}","gists_url":"https://api.github.com/users/dylanrhodes/gists{/gist_id}","starred_url":"https://api.github.com/users/dylanrhodes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dylanrhodes/subscriptions","organizations_url":"https://api.github.com/users/dylanrhodes/orgs","repos_url":"https://api.github.com/users/dylanrhodes/repos","events_url":"https://api.github.com/users/dylanrhodes/events{/privacy}","received_events_url":"https://api.github.com/users/dylanrhodes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/edx/edx-analytics-data-api/labels/waiting%20on%20author","name":"waiting on author","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-02-19T00:53:58Z","updated_at":"2015-04-20T21:47:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/edx-analytics-data-api/pulls/61","html_url":"https://github.com/edx/edx-analytics-data-api/pull/61","diff_url":"https://github.com/edx/edx-analytics-data-api/pull/61.diff","patch_url":"https://github.com/edx/edx-analytics-data-api/pull/61.patch"},"body":"This is a PR to accompany the upgraded answer_dist collection task which includes data for students' first responses to a question and is available <a href=\"https://github.com/edx/edx-analytics-pipeline/pull/73\">here</a>.  At the request of @mulby and @clinton, the API is compatible with both the old and new schemata to ensure ease of deployment.  As a result of that requirement, however, there is a fair amount of somewhat-duplicated code that should be removed in a future PR once the new answer_distribution table has been rolled out.\r\n\r\nAnother topic which probably ought to be discussed here is that the PR renames the count field to final_response_count.  This renaming was included in order to disambiguate between the usage of the new column (first_response_count) and the old column (count).  I think the new name is more appropriate; however, it should also be noted that renaming an existing column will break any existing code which relies on that field coming from the answer_distribution API.  I'm not exactly sure which approach everyone prefers - renaming the column or not - so I've implemented it this way and figured we could work it out now.  Swapping count back in for first_response_count ought to be fairly straightforward, but I think disambiguating the two count columns will be useful for the future even if it requires changes to existing code.\r\n\r\nComments and feedback are welcome on anything @mulby @clintonb @dcadams "},"comment":{"url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/comments/94577386","html_url":"https://github.com/edx/edx-analytics-data-api/pull/61#issuecomment-94577386","issue_url":"https://api.github.com/repos/edx/edx-analytics-data-api/issues/61","id":94577386,"user":{"login":"dsjen","id":5265058,"avatar_url":"https://avatars.githubusercontent.com/u/5265058?v=3","gravatar_id":"","url":"https://api.github.com/users/dsjen","html_url":"https://github.com/dsjen","followers_url":"https://api.github.com/users/dsjen/followers","following_url":"https://api.github.com/users/dsjen/following{/other_user}","gists_url":"https://api.github.com/users/dsjen/gists{/gist_id}","starred_url":"https://api.github.com/users/dsjen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dsjen/subscriptions","organizations_url":"https://api.github.com/users/dsjen/orgs","repos_url":"https://api.github.com/users/dsjen/repos","events_url":"https://api.github.com/users/dsjen/events{/privacy}","received_events_url":"https://api.github.com/users/dsjen/received_events","type":"User","site_admin":false},"created_at":"2015-04-20T21:47:50Z","updated_at":"2015-04-20T21:47:50Z","body":"@dylanrhodes Looks like it passed.  Feel free to merge.  If you don't see a merge button, please let me know and I'll click it. :)"}}