vidjil issueshttps://gitlab.inria.fr/vidjil/vidjil/-/issues2020-07-28T16:19:19+02:00https://gitlab.inria.fr/vidjil/vidjil/-/issues/4334Supprimer seg_stat ?2020-07-28T16:19:19+02:00Mathieu GiraudSupprimer seg_stat ?Dans `core/windows.cpp`, `WindowsStorage::statusToJson()` crée, dans le json, `seg_stat`, à partir de `status_by_window` :
```
"reads": [
606
],
"seg_stat": [
{
"1": 217,
"2": 389
...Dans `core/windows.cpp`, `WindowsStorage::statusToJson()` crée, dans le json, `seg_stat`, à partir de `status_by_window` :
```
"reads": [
606
],
"seg_stat": [
{
"1": 217,
"2": 389
}
```
Cela permet de connaître la répartition +/- de chaque window. Mais... est-ce que vous vous en êtes déjà servi ?
Le dernier commit mentionnant explicitement `seg_stat` est... 8991d0b29 (2014 !), même si `seg_stat` a depuis été "transporté et actualisé" par différents ~"dev-refactor".
À ce moment `seg_stat` était utilisé par ~"server-fuse". Mais... ce n'est plus le cas (pas réussi à trouver depuis quand), et désormais j'ai l'impression qu'il n'y a plus *aucune* lecture de `seg_stat` dans tout notre code (et aucun test). En attendant, cela prend 40 octets (dont beaucoup de blancs) par séquence dans le `.vidjil` (sur 1.9 Ko, voir #4036). Bref, supprimer `seg_stat` ?
On pourrait même se demander si `status_by_window` est utile... oui, sert à fixer le W50, mais est-ce tout ?
cc @duez @flothoniAlgo 2020.06