HomePhorge

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

Provenance
epriestleyAuthored on May 7 2015, 9:10 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPf3d76a90f0e3: Translate "All Day" events into the viewer's time
Branches
Unknown
Tags
Unknown

Event Timeline