{"action":"created","issue":{"url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/12","labels_url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/12/comments","events_url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/12/events","html_url":"https://github.com/MscrmTools/XrmToolBox/pull/12","id":57331926,"number":12,"title":"Loading plugins in parallel at application start up","user":{"login":"shytikov","id":1185905,"avatar_url":"https://avatars.githubusercontent.com/u/1185905?v=3","gravatar_id":"","url":"https://api.github.com/users/shytikov","html_url":"https://github.com/shytikov","followers_url":"https://api.github.com/users/shytikov/followers","following_url":"https://api.github.com/users/shytikov/following{/other_user}","gists_url":"https://api.github.com/users/shytikov/gists{/gist_id}","starred_url":"https://api.github.com/users/shytikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shytikov/subscriptions","organizations_url":"https://api.github.com/users/shytikov/orgs","repos_url":"https://api.github.com/users/shytikov/repos","events_url":"https://api.github.com/users/shytikov/events{/privacy}","received_events_url":"https://api.github.com/users/shytikov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-11T15:38:54Z","updated_at":"2015-02-11T16:03:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/MscrmTools/XrmToolBox/pulls/12","html_url":"https://github.com/MscrmTools/XrmToolBox/pull/12","diff_url":"https://github.com/MscrmTools/XrmToolBox/pull/12.diff","patch_url":"https://github.com/MscrmTools/XrmToolBox/pull/12.patch"},"body":"Simple optimization to speed-up XrmToolBox loading.\r\n\r\nI've made simple test on efficiency. I've run 10 times each approach on 24 plugins. Here is results:\r\n\r\nCase no. | Serial execution (milliseconds)| Parallel execution (milliseconds)\r\n--- | ---------------- | ------------------\r\n1 | 398 | 321\r\n2 | 384 | 313\r\n3 | 426 | 281\r\n4 | 367 | 281\r\n5 | 390 | 319\r\n6 | 429 | 301\r\n7 | 397 | 316\r\n8 | 397 | 305\r\n9 | 344 | 284\r\n10 | 329 | 326\r\n**Average** | 386.1 | 304.7\r\n**Percentage** | 100% | 78.92 %\r\n\r\n21.08% performance gain on 4 logical cores CPU.\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/comments/73906723","html_url":"https://github.com/MscrmTools/XrmToolBox/pull/12#issuecomment-73906723","issue_url":"https://api.github.com/repos/MscrmTools/XrmToolBox/issues/12","id":73906723,"user":{"login":"daryllabar","id":9728423,"avatar_url":"https://avatars.githubusercontent.com/u/9728423?v=3","gravatar_id":"","url":"https://api.github.com/users/daryllabar","html_url":"https://github.com/daryllabar","followers_url":"https://api.github.com/users/daryllabar/followers","following_url":"https://api.github.com/users/daryllabar/following{/other_user}","gists_url":"https://api.github.com/users/daryllabar/gists{/gist_id}","starred_url":"https://api.github.com/users/daryllabar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daryllabar/subscriptions","organizations_url":"https://api.github.com/users/daryllabar/orgs","repos_url":"https://api.github.com/users/daryllabar/repos","events_url":"https://api.github.com/users/daryllabar/events{/privacy}","received_events_url":"https://api.github.com/users/daryllabar/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T16:03:00Z","updated_at":"2015-02-11T16:03:00Z","body":"I like the changes, and I'm very glad you got rid of (or I'm assuming it will) the error I was getting at debug time, as well as having the plugins load correctly for the user.  All good things and they warrant your changes (I'm glad you've done them).  Just a warning to be careful about optimizing for optimizing's sake (I may be \"preaching to the choir\" here, because I don't think you made the changes to make it faster, it's just a side effect.)\r\n\r\nThe percentages aren't as important as time.  I'm gonna guess you spent 3-4 hours on this, and now it'll load .08 seconds faster, which probably isn't noticeable to the end user.  Just something to think about.  But looks good! \r\n"}}