Page MenuHomePhabricator

Improve performance of lint differencing query
AbandonedPublic

Authored by hach-que on May 28 2014, 1:22 AM.
Tags
None
Referenced Files
F15457881: D9315.diff
Sun, Mar 30, 7:47 PM
F15457847: D9315.id22101.diff
Sun, Mar 30, 7:23 PM
F15432988: D9315.diff
Mon, Mar 24, 8:43 PM
Unknown Object (File)
Feb 9 2025, 1:14 PM
Unknown Object (File)
Feb 9 2025, 1:14 PM
Unknown Object (File)
Feb 9 2025, 1:13 PM
Unknown Object (File)
Feb 9 2025, 1:13 PM
Unknown Object (File)
Feb 9 2025, 2:46 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Depends on D9233.

This substantially improves the performance of displaying lint trends in Diffusion (from 300 second query to a 5 second query).

Test Plan

Last modified information didn't take 5 minutes to load. Compared using DarkConsole.

Diff Detail

Repository
rP Phabricator
Branch
lint-fast
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 732
Build 732: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que retitled this revision from to Improve performance of lint differencing query.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Actually make the query work. Also exclude valueX where it's 0 (no change), since we want to see changes the impact the lint values.

Abandoning because Fact is very far away on the roadmap.