{"action":"created","issue":{"url":"https://api.github.com/repos/rangle/augury/issues/592","repository_url":"https://api.github.com/repos/rangle/augury","labels_url":"https://api.github.com/repos/rangle/augury/issues/592/labels{/name}","comments_url":"https://api.github.com/repos/rangle/augury/issues/592/comments","events_url":"https://api.github.com/repos/rangle/augury/issues/592/events","html_url":"https://github.com/rangle/augury/pull/592","id":174812467,"number":592,"title":"Major performance improvment by bypassing the sequence of ports when sending tree data","user":{"login":"clbond","id":1780164,"avatar_url":"https://avatars.githubusercontent.com/u/1780164?v=3","gravatar_id":"","url":"https://api.github.com/users/clbond","html_url":"https://github.com/clbond","followers_url":"https://api.github.com/users/clbond/followers","following_url":"https://api.github.com/users/clbond/following{/other_user}","gists_url":"https://api.github.com/users/clbond/gists{/gist_id}","starred_url":"https://api.github.com/users/clbond/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clbond/subscriptions","organizations_url":"https://api.github.com/users/clbond/orgs","repos_url":"https://api.github.com/users/clbond/repos","events_url":"https://api.github.com/users/clbond/events{/privacy}","received_events_url":"https://api.github.com/users/clbond/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2016-09-02T17:42:10Z","updated_at":"2016-09-07T15:41:53Z","closed_at":"2016-09-06T19:26:41Z","pull_request":{"url":"https://api.github.com/repos/rangle/augury/pulls/592","html_url":"https://github.com/rangle/augury/pull/592","diff_url":"https://github.com/rangle/augury/pull/592.diff","patch_url":"https://github.com/rangle/augury/pull/592.patch"},"body":"This PR represents a major performance improvement over previous builds.\r\n\r\nBefore this change, we were sending tree data through the normal channel. That meant every time we sent a tree or a tree delta, it had to go through this sequence:\r\n\r\nBackend script -> content script -> background channel -> frontend\r\n\r\nEach one of these steps required a serialize + deserialize (which happens in the internal Chrome code, not our code). Therefore for large trees, it was an absolute performance killer. This PR fixes that. Now the trees are just serialized and deserialized **once** on their way from the backend to the frontend.\r\n\r\nI encourage anyone who reviews this to do a side-by-side performance comparison using the Kitchen Sink stress tester or any other app."},"comment":{"url":"https://api.github.com/repos/rangle/augury/issues/comments/245322823","html_url":"https://github.com/rangle/augury/pull/592#issuecomment-245322823","issue_url":"https://api.github.com/repos/rangle/augury/issues/592","id":245322823,"user":{"login":"clbond","id":1780164,"avatar_url":"https://avatars.githubusercontent.com/u/1780164?v=3","gravatar_id":"","url":"https://api.github.com/users/clbond","html_url":"https://github.com/clbond","followers_url":"https://api.github.com/users/clbond/followers","following_url":"https://api.github.com/users/clbond/following{/other_user}","gists_url":"https://api.github.com/users/clbond/gists{/gist_id}","starred_url":"https://api.github.com/users/clbond/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clbond/subscriptions","organizations_url":"https://api.github.com/users/clbond/orgs","repos_url":"https://api.github.com/users/clbond/repos","events_url":"https://api.github.com/users/clbond/events{/privacy}","received_events_url":"https://api.github.com/users/clbond/received_events","type":"User","site_admin":false},"created_at":"2016-09-07T15:41:53Z","updated_at":"2016-09-07T15:41:53Z","body":"@oskarols Are you trying to debug an app that is running in production mode?"}}
{"actor":{"display_login":"clbond"}}