Maniphest T7815

Conpherence threads ordering messed up; perf maybe too bad now too?
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	btrahan
	Apr 13 2015, 7:31 PM

Tags

Referenced Files

None

Subscribers

Description

The ordering is likely from the application search changes by @epriestley
The perf is likely from the "A, B, C..." to "A: last message" change from @btrahan

Revisions and Commits

rP Phabricator
	D12399	rPf12839ffd414 Conpherenece - improve performance by not fetching data as often from D12347
	D12398	rPd096b9c1517e Fix Conpherence transaction ordering

Event Timeline

btrahan created this task.Apr 13 2015, 7:31 PM

btrahan raised the priority of this task from to Needs Triage.

btrahan updated the task description. (Show Details)

btrahan added projects: Conpherence, ApplicationSearch.

btrahan added subscribers: btrahan, chad, epriestley.

yeah this is 100% me

Oh, well the "upside down" part at least.

epriestley added a revision: D12398: Fix Conpherence transaction ordering.Apr 13 2015, 7:33 PM

epriestley added a commit: rPd096b9c1517e: Fix Conpherence transaction ordering.Apr 13 2015, 7:34 PM

well ?.

perfypoo diff in a sec

Looks like you've got the other half of this.

btrahan added a revision: D12399: Conpherenece - improve performance by not fetching data as often from D12347.Apr 13 2015, 7:42 PM

btrahan closed this task as Resolved by committing rPf12839ffd414: Conpherenece - improve performance by not fetching data as often from D12347.Apr 13 2015, 7:43 PM

btrahan added a commit: rPf12839ffd414: Conpherenece - improve performance by not fetching data as often from D12347.