{"action":"created","comment":{"url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/comments/58099075","id":58099075,"diff_hunk":"@@ -22,10 +22,10 @@ def setupErrorDB( hardReset = False):\n         \"CREATE TABLE status (status_id integer PRIMARY KEY, name text NOT NULL,description text NOT NULL);\",\n         \"CREATE TABLE error_type (error_type_id integer PRIMARY KEY, name text NOT NULL,description text NOT NULL);\",\n         \"CREATE SEQUENCE fileSerial START 1;\",\n-        \"CREATE TABLE file_status (file_id integer PRIMARY KEY DEFAULT nextval('fileSerial'), job_id integer, filename text, status_id integer REFERENCES status NOT NULL);\",\n+        \"CREATE TABLE file_status (file_id integer PRIMARY KEY DEFAULT nextval('fileSerial'), job_id integer, filename text, status_id integer REFERENCES status NOT NULL, headers_missing text, headers_duplicated text);\",","path":"dataactcore/scripts/setupErrorDB.py","position":5,"original_position":5,"commit_id":"1d2db593a9676f09636f3beee3a487bb742060e0","original_commit_id":"1d2db593a9676f09636f3beee3a487bb742060e0","user":{"login":"bsweger","id":540544,"avatar_url":"https://avatars.githubusercontent.com/u/540544?v=3","gravatar_id":"","url":"https://api.github.com/users/bsweger","html_url":"https://github.com/bsweger","followers_url":"https://api.github.com/users/bsweger/followers","following_url":"https://api.github.com/users/bsweger/following{/other_user}","gists_url":"https://api.github.com/users/bsweger/gists{/gist_id}","starred_url":"https://api.github.com/users/bsweger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsweger/subscriptions","organizations_url":"https://api.github.com/users/bsweger/orgs","repos_url":"https://api.github.com/users/bsweger/repos","events_url":"https://api.github.com/users/bsweger/events{/privacy}","received_events_url":"https://api.github.com/users/bsweger/received_events","type":"User","site_admin":false},"body":"The schema changes themselves should be ok, since the initialization scripts create the tables via the ORM/models. I've already updated the DB-247 branch for the updated insert statements.","created_at":"2016-03-31T18:00:08Z","updated_at":"2016-03-31T18:00:08Z","html_url":"https://github.com/fedspendingtransparency/data-act-core/pull/9#discussion_r58099075","pull_request_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9","_links":{"self":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/comments/58099075"},"html":{"href":"https://github.com/fedspendingtransparency/data-act-core/pull/9#discussion_r58099075"},"pull_request":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9"}}},"pull_request":{"url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9","id":64057761,"html_url":"https://github.com/fedspendingtransparency/data-act-core/pull/9","diff_url":"https://github.com/fedspendingtransparency/data-act-core/pull/9.diff","patch_url":"https://github.com/fedspendingtransparency/data-act-core/pull/9.patch","issue_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/9","number":9,"state":"open","locked":false,"title":"Added file information feature","user":{"login":"jworcestBAH","id":11858333,"avatar_url":"https://avatars.githubusercontent.com/u/11858333?v=3","gravatar_id":"","url":"https://api.github.com/users/jworcestBAH","html_url":"https://github.com/jworcestBAH","followers_url":"https://api.github.com/users/jworcestBAH/followers","following_url":"https://api.github.com/users/jworcestBAH/following{/other_user}","gists_url":"https://api.github.com/users/jworcestBAH/gists{/gist_id}","starred_url":"https://api.github.com/users/jworcestBAH/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jworcestBAH/subscriptions","organizations_url":"https://api.github.com/users/jworcestBAH/orgs","repos_url":"https://api.github.com/users/jworcestBAH/repos","events_url":"https://api.github.com/users/jworcestBAH/events{/privacy}","received_events_url":"https://api.github.com/users/jworcestBAH/received_events","type":"User","site_admin":false},"body":"@nmonga91 or @bsweger: These are the changes to the databases needed for the extra file information for both missing and duplicated headers and original filenames.  Sorry about the duplicate PR, made an error on the last one.","created_at":"2016-03-24T16:10:41Z","updated_at":"2016-03-31T18:00:08Z","closed_at":null,"merged_at":null,"merge_commit_sha":"595fc69b3719b76f08b5dab501ed04ffb08c6ce0","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9/commits","review_comments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9/comments","review_comment_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/comments{/number}","comments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/9/comments","statuses_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/statuses/1d2db593a9676f09636f3beee3a487bb742060e0","head":{"label":"fedspendingtransparency:added_file_information_feature","ref":"added_file_information_feature","sha":"1d2db593a9676f09636f3beee3a487bb742060e0","user":{"login":"fedspendingtransparency","id":8397762,"avatar_url":"https://avatars.githubusercontent.com/u/8397762?v=3","gravatar_id":"","url":"https://api.github.com/users/fedspendingtransparency","html_url":"https://github.com/fedspendingtransparency","followers_url":"https://api.github.com/users/fedspendingtransparency/followers","following_url":"https://api.github.com/users/fedspendingtransparency/following{/other_user}","gists_url":"https://api.github.com/users/fedspendingtransparency/gists{/gist_id}","starred_url":"https://api.github.com/users/fedspendingtransparency/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fedspendingtransparency/subscriptions","organizations_url":"https://api.github.com/users/fedspendingtransparency/orgs","repos_url":"https://api.github.com/users/fedspendingtransparency/repos","events_url":"https://api.github.com/users/fedspendingtransparency/events{/privacy}","received_events_url":"https://api.github.com/users/fedspendingtransparency/received_events","type":"Organization","site_admin":false},"repo":{"id":47700007,"name":"data-act-core","full_name":"fedspendingtransparency/data-act-core","owner":{"login":"fedspendingtransparency","id":8397762,"avatar_url":"https://avatars.githubusercontent.com/u/8397762?v=3","gravatar_id":"","url":"https://api.github.com/users/fedspendingtransparency","html_url":"https://github.com/fedspendingtransparency","followers_url":"https://api.github.com/users/fedspendingtransparency/followers","following_url":"https://api.github.com/users/fedspendingtransparency/following{/other_user}","gists_url":"https://api.github.com/users/fedspendingtransparency/gists{/gist_id}","starred_url":"https://api.github.com/users/fedspendingtransparency/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fedspendingtransparency/subscriptions","organizations_url":"https://api.github.com/users/fedspendingtransparency/orgs","repos_url":"https://api.github.com/users/fedspendingtransparency/repos","events_url":"https://api.github.com/users/fedspendingtransparency/events{/privacy}","received_events_url":"https://api.github.com/users/fedspendingtransparency/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/fedspendingtransparency/data-act-core","description":"","fork":false,"url":"https://api.github.com/repos/fedspendingtransparency/data-act-core","forks_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/forks","keys_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/teams","hooks_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/hooks","issue_events_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/events{/number}","events_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/events","assignees_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/assignees{/user}","branches_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/branches{/branch}","tags_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/tags","blobs_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/refs{/sha}","trees_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/statuses/{sha}","languages_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/languages","stargazers_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/stargazers","contributors_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/contributors","subscribers_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/subscribers","subscription_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/subscription","commits_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/commits{/sha}","git_commits_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/commits{/sha}","comments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/comments{/number}","issue_comment_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/comments{/number}","contents_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/contents/{+path}","compare_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/merges","archive_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/downloads","issues_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues{/number}","pulls_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls{/number}","milestones_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/milestones{/number}","notifications_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/labels{/name}","releases_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/releases{/id}","deployments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/deployments","created_at":"2015-12-09T15:30:31Z","updated_at":"2016-03-01T03:17:41Z","pushed_at":"2016-03-31T17:56:12Z","git_url":"git://github.com/fedspendingtransparency/data-act-core.git","ssh_url":"git@github.com:fedspendingtransparency/data-act-core.git","clone_url":"https://github.com/fedspendingtransparency/data-act-core.git","svn_url":"https://github.com/fedspendingtransparency/data-act-core","homepage":null,"size":219,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":4,"forks":0,"open_issues":4,"watchers":1,"default_branch":"master"}},"base":{"label":"fedspendingtransparency:development","ref":"development","sha":"7c4120da31ebbbfd0b3075e2cbbcc1f20ff1caf5","user":{"login":"fedspendingtransparency","id":8397762,"avatar_url":"https://avatars.githubusercontent.com/u/8397762?v=3","gravatar_id":"","url":"https://api.github.com/users/fedspendingtransparency","html_url":"https://github.com/fedspendingtransparency","followers_url":"https://api.github.com/users/fedspendingtransparency/followers","following_url":"https://api.github.com/users/fedspendingtransparency/following{/other_user}","gists_url":"https://api.github.com/users/fedspendingtransparency/gists{/gist_id}","starred_url":"https://api.github.com/users/fedspendingtransparency/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fedspendingtransparency/subscriptions","organizations_url":"https://api.github.com/users/fedspendingtransparency/orgs","repos_url":"https://api.github.com/users/fedspendingtransparency/repos","events_url":"https://api.github.com/users/fedspendingtransparency/events{/privacy}","received_events_url":"https://api.github.com/users/fedspendingtransparency/received_events","type":"Organization","site_admin":false},"repo":{"id":47700007,"name":"data-act-core","full_name":"fedspendingtransparency/data-act-core","owner":{"login":"fedspendingtransparency","id":8397762,"avatar_url":"https://avatars.githubusercontent.com/u/8397762?v=3","gravatar_id":"","url":"https://api.github.com/users/fedspendingtransparency","html_url":"https://github.com/fedspendingtransparency","followers_url":"https://api.github.com/users/fedspendingtransparency/followers","following_url":"https://api.github.com/users/fedspendingtransparency/following{/other_user}","gists_url":"https://api.github.com/users/fedspendingtransparency/gists{/gist_id}","starred_url":"https://api.github.com/users/fedspendingtransparency/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fedspendingtransparency/subscriptions","organizations_url":"https://api.github.com/users/fedspendingtransparency/orgs","repos_url":"https://api.github.com/users/fedspendingtransparency/repos","events_url":"https://api.github.com/users/fedspendingtransparency/events{/privacy}","received_events_url":"https://api.github.com/users/fedspendingtransparency/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/fedspendingtransparency/data-act-core","description":"","fork":false,"url":"https://api.github.com/repos/fedspendingtransparency/data-act-core","forks_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/forks","keys_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/teams","hooks_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/hooks","issue_events_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/events{/number}","events_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/events","assignees_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/assignees{/user}","branches_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/branches{/branch}","tags_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/tags","blobs_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/refs{/sha}","trees_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/statuses/{sha}","languages_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/languages","stargazers_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/stargazers","contributors_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/contributors","subscribers_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/subscribers","subscription_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/subscription","commits_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/commits{/sha}","git_commits_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/git/commits{/sha}","comments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/comments{/number}","issue_comment_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/comments{/number}","contents_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/contents/{+path}","compare_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/merges","archive_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/downloads","issues_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues{/number}","pulls_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls{/number}","milestones_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/milestones{/number}","notifications_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/labels{/name}","releases_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/releases{/id}","deployments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-core/deployments","created_at":"2015-12-09T15:30:31Z","updated_at":"2016-03-01T03:17:41Z","pushed_at":"2016-03-31T17:56:12Z","git_url":"git://github.com/fedspendingtransparency/data-act-core.git","ssh_url":"git@github.com:fedspendingtransparency/data-act-core.git","clone_url":"https://github.com/fedspendingtransparency/data-act-core.git","svn_url":"https://github.com/fedspendingtransparency/data-act-core","homepage":null,"size":219,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":4,"forks":0,"open_issues":4,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9"},"html":{"href":"https://github.com/fedspendingtransparency/data-act-core/pull/9"},"issue":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/9"},"comments":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/issues/9/comments"},"review_comments":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9/comments"},"review_comment":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/pulls/9/commits"},"statuses":{"href":"https://api.github.com/repos/fedspendingtransparency/data-act-core/statuses/1d2db593a9676f09636f3beee3a487bb742060e0"}}}}