Bug 32780

Summary: Use memcache to make dashboard to fast
Product: WebKit Reporter: Adam Barth <abarth>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
Patch none

Description Adam Barth 2009-12-19 19:52:23 PST
Fast is good!
Comment 1 Adam Barth 2009-12-19 19:53:21 PST
Created attachment 45255 [details]
Comment 2 WebKit Review Bot 2009-12-19 19:54:37 PST
style-queue ran check-webkit-style on attachment 45255 [details] without any errors.
Comment 3 Eric Seidel (no email) 2009-12-19 22:56:59 PST
Comment on attachment 45255 [details]

remove_api seems unrelated.

I don't see why you broke queue_name and bug_id out into local variables in UpdateStatus

Why change it to use a regexp if we compile every time?
74         regexp = re.compile("-")
 75         return regexp.sub("_", dashed_name)

replace is simpler, no?

Why do we have 3 copies of the string "attachment-summary"?  Seems error prone.

Looks OK though.
Comment 4 Adam Barth 2009-12-20 00:27:58 PST
Comment on attachment 45255 [details]

Clearing flags on attachment: 45255

Committed r52404: <http://trac.webkit.org/changeset/52404>
Comment 5 Adam Barth 2009-12-20 00:28:07 PST
All reviewed patches have been landed.  Closing bug.