WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
48500
webkit-patch suggest-reviewers -g 260550a6e30b7bf34f16bdb4a5396cf26264fc1c is very slow
https://bugs.webkit.org/show_bug.cgi?id=48500
Summary
webkit-patch suggest-reviewers -g 260550a6e30b7bf34f16bdb4a5396cf26264fc1c is...
Eric Seidel (no email)
Reported
2010-10-27 21:26:50 PDT
webkit-patch suggest-reviewers -g 260550a6e30b7bf34f16bdb4a5396cf26264fc1c is very slow
Attachments
Patch
(9.79 KB, patch)
2010-10-27 21:30 PDT
,
Eric Seidel (no email)
no flags
Details
Formatted Diff
Diff
Deploy @memoized to one more place
(10.60 KB, patch)
2010-10-27 21:48 PDT
,
Eric Seidel (no email)
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1
2010-10-27 21:30:49 PDT
Created
attachment 72137
[details]
Patch
Eric Seidel (no email)
Comment 2
2010-10-27 21:32:47 PDT
Evan (or anyone else): if you have suggestions on how to make our git log codepath faster for Git.revisions_changing_file I'd love to hear them. :)
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/common/checkout/scm.py#L676
Eric Seidel (no email)
Comment 3
2010-10-27 21:48:07 PDT
Created
attachment 72138
[details]
Deploy @memoized to one more place
Adam Barth
Comment 4
2010-10-27 21:55:43 PDT
Comment on
attachment 72138
[details]
Deploy @memoized to one more place Mind -> blown
WebKit Commit Bot
Comment 5
2010-10-27 22:12:50 PDT
Comment on
attachment 72138
[details]
Deploy @memoized to one more place Clearing flags on attachment: 72138 Committed
r70747
: <
http://trac.webkit.org/changeset/70747
>
WebKit Commit Bot
Comment 6
2010-10-27 22:12:56 PDT
All reviewed patches have been landed. Closing bug.
Evan Martin
Comment 7
2010-10-28 10:09:24 PDT
"git", "log", "--pretty=format:%H", "-%s" % limit, "--", path is the same as git rev-list HEAD -- path | head -<limit> but I expect they have the same big-O
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug