{"thread":{"id":"9705","subject":"gitk startup time improvement","startedAt":"2007-08-30T12:01:13Z","lastAt":"2007-08-30T20:42:22Z","messageCount":4,"participants":["Paul Mackerras","David Tweed","Bart Trojanowski","Alex Riesen"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"51900","messageId":"18134.45449.527540.940620@cargo.ozlabs.ibm.com","threadId":"9705","inReplyTo":null,"subject":"gitk startup time improvement","fromName":"Paul Mackerras","fromEmail":"paulus@samba.org","sentAt":"2007-08-30T12:01:13Z","receivedAt":"2007-08-30T12:01:13Z","isPatch":false,"sender":{"key":"paulus@samba.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"In the 'dev' branch of the gitk.git repository at\n\ngit://git.kernel.org/pub/scm/gitk/gitk.git\n\nthere is now a version of gitk that is much faster at starting up than\nthe standard version.  Firstly, it doesn't lay out the whole graph,\nonly the parts that get displayed (plus a little bit either side), and\nsecondly, it caches the topology information that is used to determine\nwhich tags a commit precedes and follows, and which branches it is on.\n\nI'd like people to try it.  Comments, bug reports, patches etc. are\nwelcome, of course.\n\nPaul.\n"},{"id":"51916","messageId":"e1dab3980708300816l5fee87a1pade9701c1bfb165@mail.gmail.com","threadId":"9705","inReplyTo":"18134.45449.527540.940620@cargo.ozlabs.ibm.com","subject":"Re: gitk startup time improvement","fromName":"David Tweed","fromEmail":"david.tweed@gmail.com","sentAt":"2007-08-30T15:16:53Z","receivedAt":"2007-08-30T15:16:53Z","isPatch":false,"sender":{"key":"david.tweed@gmail.com","avatar":null},"body":"On 8/30/07, Paul Mackerras <paulus@samba.org> wrote:\n> I'd like people to try it.  Comments, bug reports, patches etc. are\n> welcome, of course.\n\nMy main repo is too small to show performance issues as of the gitk\nversion included in 1.5.3.rc5, but FWIW the startup times for both\n1.5.3.rc5 & your branch are both less than 1s (with warm cache).\n\n-- \ncheers, dave tweed__________________________\ndavid.tweed@gmail.com\nRm 124, School of Systems Engineering, University of Reading.\n\"we had no idea that when we added templates we were adding a Turing-\ncomplete compile-time language.\" -- C++ standardisation committee\n"},{"id":"51928","messageId":"20070830164202.GR10772@jukie.net","threadId":"9705","inReplyTo":"18134.45449.527540.940620@cargo.ozlabs.ibm.com","subject":"Re: gitk startup time improvement","fromName":"Bart Trojanowski","fromEmail":"bart@jukie.net","sentAt":"2007-08-30T16:42:02Z","receivedAt":"2007-08-30T16:42:02Z","isPatch":false,"sender":{"key":"bart@jukie.net","avatar":"https://avatars.githubusercontent.com/u/6721?v=4"},"body":"* Paul Mackerras <paulus@samba.org> [070830 08:01]:\n> I'd like people to try it.  Comments, bug reports, patches etc. are\n> welcome, of course.\n\nI am getting about a 4x improvement (1s vs 4s) running the 'dev' gitk on\nthe kernel tree, with warm cache.\n\n-Bart\n\n-- \n\t\t\t\tWebSig: http://www.jukie.net/~bart/sig/\n"},{"id":"51939","messageId":"20070830204222.GB15405@steel.home","threadId":"9705","inReplyTo":"18134.45449.527540.940620@cargo.ozlabs.ibm.com","subject":"Re: gitk startup time improvement","fromName":"Alex Riesen","fromEmail":"raa.lkml@gmail.com","sentAt":"2007-08-30T20:42:22Z","receivedAt":"2007-08-30T20:42:22Z","isPatch":false,"sender":{"key":"raa.lkml@gmail.com","avatar":"https://avatars.githubusercontent.com/u/324101?v=4"},"body":"Paul Mackerras, Thu, Aug 30, 2007 14:01:13 +0200:\n> In the 'dev' branch of the gitk.git repository at\n> \n> git://git.kernel.org/pub/scm/gitk/gitk.git\n> \n> there is now a version of gitk that is much faster at starting up than\n> the standard version.  Firstly, it doesn't lay out the whole graph,\n> only the parts that get displayed (plus a little bit either side), and\n> secondly, it caches the topology information that is used to determine\n> which tags a commit precedes and follows, and which branches it is on.\n> \n> I'd like people to try it.  Comments, bug reports, patches etc. are\n> welcome, of course.\n\nWell, it works. The improvements are not very noticeable, though (I\ntried git and kernel repos). Besides, there is a slight delay (~1-2\nsec) in kernel repo (writing the cache out?) on exit.\n\nI used gitk from this commit:\n\n    commit 5cd15b6b7f87dc61f729ad31a682ffc394560273\n    Author: Paul Mackerras <paulus@samba.org>\n    Date:   Thu Aug 30 21:54:17 2007 +1000\n\n\tgitk: Add a cache for the topology info\n\nBTW, what should happen if I try running many gitks in the same repo?\nIs the cache safe for concurrent writing by them? Is it NFS-safe?\n"}]}