Linus Torvalds wrote:
quoted text
> On Sat, 27 Jan 2007, Shawn O. Pearce wrote:
>> _THIS_ is worth doing.  I've been having a lot of discussion on
>> #git with Simon 'corecode' Schubert and Chris Lee about how poorly
>> git-blame performs compared to its counterpart in Subversion.
> I think we're *much* better off trying to get people off the "git-blame=
"=20
quoted text
> mentality entirely.
>=20
> Don't screw up git in trying to make "git blame" performance better.

Okay, let's try to assume for now that nobody said "git blame".  Instead =
let's say:

git rev-list and git log (with or without -p) perform poorly when invoked=
 with a pathspec.

quoted text
>    Which implies that with a slow CPU, and a really humongous archive, =
it=20
quoted text
>    will take much more. Is five seconds slow enough that people think i=
t=20
quoted text
>    is slow? Yes. Is 30 seconds approaching painful? Yes. But you should=
=20
quoted text
>    try to aim for really just telling people that it's not a common=20
>    operation.

I agreee with those numbers.  However, on a converted KDE repo, they are =
*completely* different:

git log kdelibs/README takes 1:18.  One minute, eighteen seconds.
git rev-list and git blame take roughly the same time.

This particular file has 64 revisions.  However there are ~ 375000 revisi=
ons in the converted repo.

My and also Shawn's point was not about the speed of git blame itself.  I=
t is about pathspec/rev operations.  The operation time does not scale wi=
th the number of changes to the file/object/call-it-whatever, but with th=
e number of total commits in the branch.

That's what we were getting at.  Not the superiority of git blame (no iro=
ny) and thus reduced speed, but the algorithmic deficiency of any operati=
on on a pathspec/object, which can be easily fixed.

cheers
  simon

--=20
Serve - BSD     +++  RENT this banner advert  +++    ASCII Ribbon   /"\
Work - Mac      +++  space for low =E2=82=AC=E2=82=AC=E2=82=AC NOW!1  +++=
      Campaign     \ /
Party Enjoy Relax   |   http://dragonflybsd.org      Against  HTML   \
Dude 2c 2 the max   !   http://golden-apple.biz       Mail + News   / \