668216 - Add new entries to graph server db for codesighs replacement

Status: RESOLVED WONTFIX

(Webtools Graveyard :: Graph Server, defect)

Description

[(not currently active) Ted Mielczarek]

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.

Comment 1

[alice nodelman [:alice] [:anode]]

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);

Comment 2

[Chris Cooper [:coop] (he/him)]

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);

Comment 3

[(not currently active) Ted Mielczarek]

Sold!

Comment 4

[Ed Morley [:emorley]]

Will the new codesize reduce end-to-end times?

Comment 5

[(not currently active) Ted Mielczarek]

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

[Ed Morley [:emorley]]

Oh of course, bug 803736 removed the old one (I'm even CCed on the bug), memory fail :-)

Comment 7

[Phil Ringnalda (:philor)]

Hahaha, codesize.