{"action":"created","issue":{"url":"https://api.github.com/repos/influxdata/influxdb/issues/7203","repository_url":"https://api.github.com/repos/influxdata/influxdb","labels_url":"https://api.github.com/repos/influxdata/influxdb/issues/7203/labels{/name}","comments_url":"https://api.github.com/repos/influxdata/influxdb/issues/7203/comments","events_url":"https://api.github.com/repos/influxdata/influxdb/issues/7203/events","html_url":"https://github.com/influxdata/influxdb/pull/7203","id":172920014,"number":7203,"title":"Ensure we don't mutate provided statistics tags","user":{"login":"e-dard","id":501993,"avatar_url":"https://avatars.githubusercontent.com/u/501993?v=3","gravatar_id":"","url":"https://api.github.com/users/e-dard","html_url":"https://github.com/e-dard","followers_url":"https://api.github.com/users/e-dard/followers","following_url":"https://api.github.com/users/e-dard/following{/other_user}","gists_url":"https://api.github.com/users/e-dard/gists{/gist_id}","starred_url":"https://api.github.com/users/e-dard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/e-dard/subscriptions","organizations_url":"https://api.github.com/users/e-dard/orgs","repos_url":"https://api.github.com/users/e-dard/repos","events_url":"https://api.github.com/users/e-dard/events{/privacy}","received_events_url":"https://api.github.com/users/e-dard/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-08-24T10:42:28Z","updated_at":"2016-08-24T15:37:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdata/influxdb/pulls/7203","html_url":"https://github.com/influxdata/influxdb/pull/7203","diff_url":"https://github.com/influxdata/influxdb/pull/7203.diff","patch_url":"https://github.com/influxdata/influxdb/pull/7203.patch"},"body":"###### Required for all non-trivial PRs\r\n- [ ] Rebased/mergable\r\n- [ ] Tests pass\r\n\r\n@jsternberg as we discussed yesterday."},"comment":{"url":"https://api.github.com/repos/influxdata/influxdb/issues/comments/242110038","html_url":"https://github.com/influxdata/influxdb/pull/7203#issuecomment-242110038","issue_url":"https://api.github.com/repos/influxdata/influxdb/issues/7203","id":242110038,"user":{"login":"e-dard","id":501993,"avatar_url":"https://avatars.githubusercontent.com/u/501993?v=3","gravatar_id":"","url":"https://api.github.com/users/e-dard","html_url":"https://github.com/e-dard","followers_url":"https://api.github.com/users/e-dard/followers","following_url":"https://api.github.com/users/e-dard/following{/other_user}","gists_url":"https://api.github.com/users/e-dard/gists{/gist_id}","starred_url":"https://api.github.com/users/e-dard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/e-dard/subscriptions","organizations_url":"https://api.github.com/users/e-dard/orgs","repos_url":"https://api.github.com/users/e-dard/repos","events_url":"https://api.github.com/users/e-dard/events{/privacy}","received_events_url":"https://api.github.com/users/e-dard/received_events","type":"User","site_admin":false},"created_at":"2016-08-24T15:37:25Z","updated_at":"2016-08-24T15:37:25Z","body":"@jsternberg you mean `MergeTags(a, b map[string]string)`?\r\n\r\nI had a couple of reasons for the `StatisticTags` type:\r\n\r\n - It's impossible to get the arguments the wrong way around and accidentally merge the wrong thing onto the wrong thing;\r\n - It makes it clear what sort of tags we're talking about when we do `defaultTags models.StatisticTags` on the types, rather than `defaultTags map[string]string`.\r\n\r\nI don't mind changing it to  `MergeTags(a, b map[string]string)` if it's preferred though."}}
{"actor":{"display_login":"e-dard"}}