Walter Bright wrote:I haven't seen this in the development of git, although to be fair, you didn't mention the number of developers that were simultaneously working on your project. If it was you alone, I can imagine you were reluctant to change it just to see if something is faster. Opensource projects with many contributors (git, linux) work differently, since one or a few among the plethora of authors will almost always be a true expert at the problem being solved. The current pack-format and how it's read is one such example. It was done once, by the combined efforts of Linus and Junio (this is all off the top of my head and I cba to go looking up the details, so bear with me if there are errors). Linus and Junio are both very good C-programmers, but the handling of packfiles was not what you'd call their specialty. Along came Nicolas Pitre, another excellent C programmer, who probably has done some similar work before. He constructed a better algorithm, eventually resulting in the ultimate performance win with a net gain in both time and size (gj, Nicolas). The point is that, given enough developers, *someone* is bound to find an algorithm that works so well that it's no longer worth investing time to even discuss if anything else would work better, either because it moves the performance bottleneck to somewhere else (where further speedups would no longer produce humanly measurable improvements), or because the action seems instantanous to the user (further improvements simply aren't worth it, because no valuable resource will be saved from it). -- Andreas Ericsson andreas.ericsson@op5.se OP5 AB www.op5.se Tel: +46 8-230225 Fax: +46 8-230231 - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
| Anders | PROBLEM: high load average when idle |
| Rafael J. Wysocki | [Bug #11264] Invalid op opcode in kernel/workqueue |
| Bart Van Assche | Integration of SCST in the mainstream Linux kernel |
| Andrew Morton | Re: [PATCH 2/2][MTD] Add support for > 2GiB MTD devices |
git: | |
| Pietro Mascagni | GIT vs Other: Need argument |
| koreth | [PATCH] Change to the repository's root directory if needed. |
| Andreas Hildebrandt | CVS-$Id:$ replacement in git? |
| Mike Gant | git clone operation |
| Stefan Beke | mail dovecot: pipe() failed: Too many open files |
| Richard Stallman | Real men don't attack straw men |
| GVG GVG | ssh_exchange_identification: Connection closed by remote host |
| Karsten McMinn | Re: Packets Per Second Limit? |
| Douglas Muir | Re: Linux on Macintrashes? (Was: Re: Linux on |
| Jim Winstead Jr. | Re: Root Disk/Book Disk Compatibility |
| Mathew B Beall | Installation |
| Stephen Tweedie | Re: No utmp entry. You must exec "login" from lowest level "sh" |
