HomePhabricator

Scope "in-flight" PHIDs more carefully

Description

Scope "in-flight" PHIDs more carefully

Fixes T8125. In Feed, we query for a bunch of objects and also a bunch of transactions.

The transactions require the objects. Normally, whichever executes last will fill out of the Workspace cheaply, so this query strategy is fine overall.

The new "in-flight" code would mark everything in flight before the transactions loaded, though, so they'd fail to load even though the query plan is not cyclic.

Instead, be more surgical and mark things in flight only immediately before we put them in flight.

Test Plan: Feed now shows more stories again; files with cycles still load in finite time.

Auditors: btrahan

Details

Auditors
btrahan
Group Auditors
Cowboy Commits
Provenance
epriestleyAuthored on
epriestleyPushed on May 8 2015, 4:14 AM
Parents
rPf3d76a90f0e3: Translate "All Day" events into the viewer's time
Branches
Unknown
Tags
Unknown
Tasks
T8125: Recent Activity is not recent after phabricator upgrade