{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto/issues/5787","repository_url":"https://api.github.com/repos/prestodb/presto","labels_url":"https://api.github.com/repos/prestodb/presto/issues/5787/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto/issues/5787/comments","events_url":"https://api.github.com/repos/prestodb/presto/issues/5787/events","html_url":"https://github.com/prestodb/presto/pull/5787","id":169208406,"number":5787,"title":"Add user field to verifier config to store who ran the verifier in the results output","user":{"login":"carroux","id":5157761,"avatar_url":"https://avatars.githubusercontent.com/u/5157761?v=3","gravatar_id":"","url":"https://api.github.com/users/carroux","html_url":"https://github.com/carroux","followers_url":"https://api.github.com/users/carroux/followers","following_url":"https://api.github.com/users/carroux/following{/other_user}","gists_url":"https://api.github.com/users/carroux/gists{/gist_id}","starred_url":"https://api.github.com/users/carroux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carroux/subscriptions","organizations_url":"https://api.github.com/users/carroux/orgs","repos_url":"https://api.github.com/users/carroux/repos","events_url":"https://api.github.com/users/carroux/events{/privacy}","received_events_url":"https://api.github.com/users/carroux/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/prestodb/presto/labels/CLA%20Signed","name":"CLA Signed","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"cberner","id":842688,"avatar_url":"https://avatars.githubusercontent.com/u/842688?v=3","gravatar_id":"","url":"https://api.github.com/users/cberner","html_url":"https://github.com/cberner","followers_url":"https://api.github.com/users/cberner/followers","following_url":"https://api.github.com/users/cberner/following{/other_user}","gists_url":"https://api.github.com/users/cberner/gists{/gist_id}","starred_url":"https://api.github.com/users/cberner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cberner/subscriptions","organizations_url":"https://api.github.com/users/cberner/orgs","repos_url":"https://api.github.com/users/cberner/repos","events_url":"https://api.github.com/users/cberner/events{/privacy}","received_events_url":"https://api.github.com/users/cberner/received_events","type":"User","site_admin":false},"assignees":[{"login":"cberner","id":842688,"avatar_url":"https://avatars.githubusercontent.com/u/842688?v=3","gravatar_id":"","url":"https://api.github.com/users/cberner","html_url":"https://github.com/cberner","followers_url":"https://api.github.com/users/cberner/followers","following_url":"https://api.github.com/users/cberner/following{/other_user}","gists_url":"https://api.github.com/users/cberner/gists{/gist_id}","starred_url":"https://api.github.com/users/cberner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cberner/subscriptions","organizations_url":"https://api.github.com/users/cberner/orgs","repos_url":"https://api.github.com/users/cberner/repos","events_url":"https://api.github.com/users/cberner/events{/privacy}","received_events_url":"https://api.github.com/users/cberner/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-08-03T19:05:19Z","updated_at":"2016-08-03T19:23:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/prestodb/presto/pulls/5787","html_url":"https://github.com/prestodb/presto/pull/5787","diff_url":"https://github.com/prestodb/presto/pull/5787.diff","patch_url":"https://github.com/prestodb/presto/pull/5787.patch"},"body":"Having this field stored with the verifier results will be useful for distinguishing between ad hoc verifier runs and those triggered automatically. It will also be useful for users who wish to filter results to only see those from verifier runs they triggered.\r\n\r\nI modified existing unit tests to cover my changes and built then tested the verifier with my changes, and confirmed that the name of the user owning the verifier process was being stored with the results correctly."},"comment":{"url":"https://api.github.com/repos/prestodb/presto/issues/comments/237343715","html_url":"https://github.com/prestodb/presto/pull/5787#issuecomment-237343715","issue_url":"https://api.github.com/repos/prestodb/presto/issues/5787","id":237343715,"user":{"login":"cberner","id":842688,"avatar_url":"https://avatars.githubusercontent.com/u/842688?v=3","gravatar_id":"","url":"https://api.github.com/users/cberner","html_url":"https://github.com/cberner","followers_url":"https://api.github.com/users/cberner/followers","following_url":"https://api.github.com/users/cberner/following{/other_user}","gists_url":"https://api.github.com/users/cberner/gists{/gist_id}","starred_url":"https://api.github.com/users/cberner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cberner/subscriptions","organizations_url":"https://api.github.com/users/cberner/orgs","repos_url":"https://api.github.com/users/cberner/repos","events_url":"https://api.github.com/users/cberner/events{/privacy}","received_events_url":"https://api.github.com/users/cberner/received_events","type":"User","site_admin":false},"created_at":"2016-08-03T19:23:12Z","updated_at":"2016-08-03T19:23:12Z","body":"It looks like this is just passed through for logging purposes, would it make sense to have our Python wrapper just log this separately instead?"}}
{"actor":{"display_login":"cberner"}}