{"action":"created","issue":{"url":"https://api.github.com/repos/webpack/webpack/issues/2210","repository_url":"https://api.github.com/repos/webpack/webpack","labels_url":"https://api.github.com/repos/webpack/webpack/issues/2210/labels{/name}","comments_url":"https://api.github.com/repos/webpack/webpack/issues/2210/comments","events_url":"https://api.github.com/repos/webpack/webpack/issues/2210/events","html_url":"https://github.com/webpack/webpack/pull/2210","id":142075929,"number":2210,"title":"Add a lookup index for modules to Chunks.","user":{"login":"blowery","id":14350,"avatar_url":"https://avatars.githubusercontent.com/u/14350?v=3","gravatar_id":"","url":"https://api.github.com/users/blowery","html_url":"https://github.com/blowery","followers_url":"https://api.github.com/users/blowery/followers","following_url":"https://api.github.com/users/blowery/following{/other_user}","gists_url":"https://api.github.com/users/blowery/gists{/gist_id}","starred_url":"https://api.github.com/users/blowery/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blowery/subscriptions","organizations_url":"https://api.github.com/users/blowery/orgs","repos_url":"https://api.github.com/users/blowery/repos","events_url":"https://api.github.com/users/blowery/events{/privacy}","received_events_url":"https://api.github.com/users/blowery/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-03-19T15:51:45Z","updated_at":"2016-05-09T14:10:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/webpack/webpack/pulls/2210","html_url":"https://github.com/webpack/webpack/pull/2210","diff_url":"https://github.com/webpack/webpack/pull/2210.diff","patch_url":"https://github.com/webpack/webpack/pull/2210.patch"},"body":"Since Chunks often contain many many modules in large projects, being able to determine if a module is part of a chunk efficiently is important. This patch adds a lookup index off of the debugId which users of the chunk can use to determine if a module is in a chunk.\r\n\r\nThis patch also teaches the RemoveParentModulePlugin to use the index. In my profiling, that particular lookup is responsible for 99% of the time spent in the plugin, which is responsible for 95% of the rebuild time in large projects.\r\n\r\nI did not apply this technique elsewhere, but it could be useful in many other places across the codebase. Adding a new `contains` method to collections that uses the index if available would make future changes easier to apply."},"comment":{"url":"https://api.github.com/repos/webpack/webpack/issues/comments/217874967","html_url":"https://github.com/webpack/webpack/pull/2210#issuecomment-217874967","issue_url":"https://api.github.com/repos/webpack/webpack/issues/2210","id":217874967,"user":{"login":"blowery","id":14350,"avatar_url":"https://avatars.githubusercontent.com/u/14350?v=3","gravatar_id":"","url":"https://api.github.com/users/blowery","html_url":"https://github.com/blowery","followers_url":"https://api.github.com/users/blowery/followers","following_url":"https://api.github.com/users/blowery/following{/other_user}","gists_url":"https://api.github.com/users/blowery/gists{/gist_id}","starred_url":"https://api.github.com/users/blowery/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blowery/subscriptions","organizations_url":"https://api.github.com/users/blowery/orgs","repos_url":"https://api.github.com/users/blowery/repos","events_url":"https://api.github.com/users/blowery/events{/privacy}","received_events_url":"https://api.github.com/users/blowery/received_events","type":"User","site_admin":false},"created_at":"2016-05-09T14:10:20Z","updated_at":"2016-05-09T14:10:20Z","body":"> Could you add a check for debugId availability and fallback to the old check?\r\n\r\n@sokra  Done, just waiting on the build"}}