Add new entries to graph server db for codesighs replacement

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
7 years ago
2 years ago

People

(Reporter: ted, Unassigned)

Tracking

Dependency tree / graph

Details

I don't know what we want to call these, but in bug 659950 I landed a patch that's going to produce two new metrics for us:
1) size of the installer package, in bytes
2) size of the installed application, in bytes

My plan is to replace the existing codesighs metric (Z) with these two new metrics. We should use a new name for #1 because it won't match the existing codesighs metric anyway, and if we make a new name we can backfill data for it if we so desire.
Blocks: 668219
Here's the current codesighs entries in the db:

insert into tests values (NULL,"codesighs","Codesighs",0,1,NULL);
insert into tests values (NULL,"codesighs_embed","Codesighs (embed)",0,1,NULL);
I propose:

insert into tests values (NULL,"codesize","Code size (bytes)",0,1,NULL);
insert into tests values (NULL,"installersize","Installer size (bytes)",0,1,NULL);
Sold!

Comment 4

6 years ago
Will the new codesize reduce end-to-end times?
Unlikely, given that we stopped running the old one. The new one is unlikely to have any noticeable impact though, since it just stats some files.

Comment 6

6 years ago
Oh of course, bug 803736 removed the old one (I'm even CCed on the bug), memory fail :-)
Hahaha, codesize.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

2 years ago
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.