{"action":"created","issue":{"url":"https://api.github.com/repos/emberjs/data/issues/3854","labels_url":"https://api.github.com/repos/emberjs/data/issues/3854/labels{/name}","comments_url":"https://api.github.com/repos/emberjs/data/issues/3854/comments","events_url":"https://api.github.com/repos/emberjs/data/issues/3854/events","html_url":"https://github.com/emberjs/data/pull/3854","id":111320589,"number":3854,"title":"Merge `normalizeRelationships` and `setupRelationships` methods in store","user":{"login":"trentmwillis","id":3526753,"avatar_url":"https://avatars.githubusercontent.com/u/3526753?v=3","gravatar_id":"","url":"https://api.github.com/users/trentmwillis","html_url":"https://github.com/trentmwillis","followers_url":"https://api.github.com/users/trentmwillis/followers","following_url":"https://api.github.com/users/trentmwillis/following{/other_user}","gists_url":"https://api.github.com/users/trentmwillis/gists{/gist_id}","starred_url":"https://api.github.com/users/trentmwillis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trentmwillis/subscriptions","organizations_url":"https://api.github.com/users/trentmwillis/orgs","repos_url":"https://api.github.com/users/trentmwillis/repos","events_url":"https://api.github.com/users/trentmwillis/events{/privacy}","received_events_url":"https://api.github.com/users/trentmwillis/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-14T04:14:16Z","updated_at":"2015-10-15T22:17:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/emberjs/data/pulls/3854","html_url":"https://github.com/emberjs/data/pull/3854","diff_url":"https://github.com/emberjs/data/pull/3854.diff","patch_url":"https://github.com/emberjs/data/pull/3854.patch"},"body":"While working on some performance related tasks, I noticed that `normalizeRelationships` and `setupRelationships` both loop over the relationships for a record. The only time these functions are used is in sequence, so it looks like these could be merged to cut down on some time complexity.\r\n\r\nWill be trying this out in our application tomorrow to see if it has any side effects, but it doesn't appear to so far."},"comment":{"url":"https://api.github.com/repos/emberjs/data/issues/comments/148538871","html_url":"https://github.com/emberjs/data/pull/3854#issuecomment-148538871","issue_url":"https://api.github.com/repos/emberjs/data/issues/3854","id":148538871,"user":{"login":"tchak","id":12428,"avatar_url":"https://avatars.githubusercontent.com/u/12428?v=3","gravatar_id":"","url":"https://api.github.com/users/tchak","html_url":"https://github.com/tchak","followers_url":"https://api.github.com/users/tchak/followers","following_url":"https://api.github.com/users/tchak/following{/other_user}","gists_url":"https://api.github.com/users/tchak/gists{/gist_id}","starred_url":"https://api.github.com/users/tchak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tchak/subscriptions","organizations_url":"https://api.github.com/users/tchak/orgs","repos_url":"https://api.github.com/users/tchak/repos","events_url":"https://api.github.com/users/tchak/events{/privacy}","received_events_url":"https://api.github.com/users/tchak/received_events","type":"User","site_admin":false},"created_at":"2015-10-15T22:17:04Z","updated_at":"2015-10-15T22:17:04Z","body":"LGTM :+1:"}}