{"action":"created","issue":{"url":"https://api.github.com/repos/BobBuildTool/bob/issues/205","repository_url":"https://api.github.com/repos/BobBuildTool/bob","labels_url":"https://api.github.com/repos/BobBuildTool/bob/issues/205/labels{/name}","comments_url":"https://api.github.com/repos/BobBuildTool/bob/issues/205/comments","events_url":"https://api.github.com/repos/BobBuildTool/bob/issues/205/events","html_url":"https://github.com/BobBuildTool/bob/pull/205","id":331931396,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk0NTM3MTUx","number":205,"title":"build: do not generate audit trail in dev mode","user":{"login":"rhubert","id":937126,"node_id":"MDQ6VXNlcjkzNzEyNg==","avatar_url":"https://avatars2.githubusercontent.com/u/937126?v=4","gravatar_id":"","url":"https://api.github.com/users/rhubert","html_url":"https://github.com/rhubert","followers_url":"https://api.github.com/users/rhubert/followers","following_url":"https://api.github.com/users/rhubert/following{/other_user}","gists_url":"https://api.github.com/users/rhubert/gists{/gist_id}","starred_url":"https://api.github.com/users/rhubert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhubert/subscriptions","organizations_url":"https://api.github.com/users/rhubert/orgs","repos_url":"https://api.github.com/users/rhubert/repos","events_url":"https://api.github.com/users/rhubert/events{/privacy}","received_events_url":"https://api.github.com/users/rhubert/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-06-13T10:18:22Z","updated_at":"2018-06-21T09:15:27Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/BobBuildTool/bob/pulls/205","html_url":"https://github.com/BobBuildTool/bob/pull/205","diff_url":"https://github.com/BobBuildTool/bob/pull/205.diff","patch_url":"https://github.com/BobBuildTool/bob/pull/205.patch"},"body":"In dev mode a common used scenario is to rebuild after a small code\r\nchange. For this generating the audit trail may take a significant amount of\r\ntime or even more time compared to the real compile time.\r\n\r\nFor a large module with some dependencies a\r\n\r\n ```bob dev -b <module>```\r\n\r\ntakes 1:15min with audit trail and 0:30min without after modifying a\r\nsingle line of code.\r\n\r\nThis change disable the audit trail for 'bob dev' unless upload is\r\nenabled and adds some controll switches."},"comment":{"url":"https://api.github.com/repos/BobBuildTool/bob/issues/comments/399033712","html_url":"https://github.com/BobBuildTool/bob/pull/205#issuecomment-399033712","issue_url":"https://api.github.com/repos/BobBuildTool/bob/issues/205","id":399033712,"node_id":"MDEyOklzc3VlQ29tbWVudDM5OTAzMzcxMg==","user":{"login":"jkloetzke","id":234300,"node_id":"MDQ6VXNlcjIzNDMwMA==","avatar_url":"https://avatars0.githubusercontent.com/u/234300?v=4","gravatar_id":"","url":"https://api.github.com/users/jkloetzke","html_url":"https://github.com/jkloetzke","followers_url":"https://api.github.com/users/jkloetzke/followers","following_url":"https://api.github.com/users/jkloetzke/following{/other_user}","gists_url":"https://api.github.com/users/jkloetzke/gists{/gist_id}","starred_url":"https://api.github.com/users/jkloetzke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jkloetzke/subscriptions","organizations_url":"https://api.github.com/users/jkloetzke/orgs","repos_url":"https://api.github.com/users/jkloetzke/repos","events_url":"https://api.github.com/users/jkloetzke/events{/privacy}","received_events_url":"https://api.github.com/users/jkloetzke/received_events","type":"User","site_admin":false},"created_at":"2018-06-21T09:15:26Z","updated_at":"2018-06-21T09:15:26Z","author_association":"MEMBER","body":"That's still a 7s overhead which is more than expected. I guess this can be further optimized by storing the audit trail parts in a sqlite database instead. But that is a bit more involved and may be worth a separete MR.\r\n\r\nShould I merge it or is there still some testing needed?"}}
{"actor":{"display_login":"jkloetzke"}}