Page MenuHomePhabricator
Differential D11808

Only increment status message cursor if we're going to consume the message
ClosedPublic
Actions

Authored by epriestley on Feb 18 2015, 8:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 29, 3:23 PM
Unknown Object (File)
Thu, Apr 24, 12:34 PM
Unknown Object (File)
Wed, Apr 23, 6:19 PM
Unknown Object (File)
Wed, Apr 23, 11:24 AM
Unknown Object (File)
Tue, Apr 22, 6:51 PM
Unknown Object (File)
Mon, Apr 21, 1:15 PM
Unknown Object (File)
Mar 23 2025, 2:42 PM
Unknown Object (File)
Mar 22 2025, 8:45 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rPdd969673060f: Only increment status message cursor if we're going to consume the message
Summary

Fixes the long uptake we saw on meta.phacility.com. I regressed this in D11795.

We make three calls to this method, but only one actually consumes the messages. The other two are just checking to see if there are any messages.

Only move the cursor up if we're actually going to process the messages.

Test Plan

Sort of tricky to test convincingly since it's inherently race-prone, but ran debug pulllocal and pushed update messages and saw it pick them up.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 28465.Feb 18 2015, 8:47 PM
epriestley retitled this revision from to Only increment status message cursor if we're going to consume the message.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
Herald added a subscriber: epriestley. · View Herald TranscriptFeb 18 2015, 8:47 PM
btrahan accepted this revision.Feb 18 2015, 8:52 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 18 2015, 8:52 PM
Closed by commit rPdd969673060f: Only increment status message cursor if we're going to consume the message (authored by epriestley, committed by epriestley). · Explain WhyFeb 18 2015, 8:53 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
daemon/
23 lines

Diff 28466

View Options

src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php

Loading...
Phabricator · Built by Phacility