Homec4science

Conpherence - fix a fatal

Authored by Bob Trahan <btrahan@phacility.com> on Apr 10 2015, 18:08.

Description

Conpherence - fix a fatal

Summary:
Ref T7795.

I can't get this to reproduce and its confusing to me how its possible. The trace in T7795 uses the "LOAD" pathway on the update controller. Under the hood, this issues a ThreadQuery with needTransactions to true. With needTransactions to true, the transactions and pertinent handles are all loaded nicely.

So... best guess is there has been some LIMIT of transactions since the offending person participated...? Alternative fix which would probably work is to specify needParticipantCache to true.

More on T7795 - the user report found the "a, b, c..." subtitle thing in the messages dropdown confusing. Yet another fix here would be to change that to be something like "a: snippet of what a said...". I'll discuss that on the task.

Test Plan: iiam

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: nevogd, Korvin, epriestley

Maniphest Tasks: T7795

Differential Revision: https://secure.phabricator.com/D12336

Details

Committed
Bob Trahan <btrahan@phacility.com>Apr 10 2015, 18:08
Pushed
aubortJan 31 2017, 17:16
Parents
rPHd44f05eead96: Clean up some more handle callsites
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH26f7b69ab2f4: Conpherence - fix a fatal (authored by Bob Trahan <btrahan@phacility.com>).Apr 10 2015, 18:08