Page MenuHomePhabricator

Reduce parse latency for changes pushed to hosted repositories
ClosedPublic

Authored by epriestley on Jan 17 2014, 9:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 9, 11:11 AM
Unknown Object (File)
Sun, Feb 9, 11:11 AM
Unknown Object (File)
Sun, Feb 9, 11:11 AM
Unknown Object (File)
Sun, Feb 9, 11:11 AM
Unknown Object (File)
Mon, Feb 3, 8:50 AM
Unknown Object (File)
Mon, Feb 3, 8:49 AM
Unknown Object (File)
Thu, Jan 23, 9:48 PM
Unknown Object (File)
Thu, Jan 23, 9:47 PM
Subscribers
Tokens
"Like" token, awarded by dctrwatson.

Details

Summary

Currently, we can sit in a sleep() for up to 15 seconds (or longer, in some cases), even if there's a parse request.

Try polling the DB every second to see if we should wake up instead. This might or might not produce significant improvements, but seems OK to try and see.

Also a small fix for logging branch names with % in them, etc.

Test Plan

Ran phd debug pulllocal and pushed commits, saw them parse within a second.

Diff Detail

Branch
cparser2.x
Lint
Lint Passed
Unit
Tests Passed