Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
vidjil
vidjil
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,696
    • Issues 1,696
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 88
    • Merge Requests 88
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • vidjil
  • vidjilvidjil
  • Issues
  • #4334

Closed
Open
Opened Jun 11, 2020 by Mathieu Giraud@magiraudOwner

Supprimer 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
        }

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... 8991d0b2 (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 @flothoni

To upload designs, you'll need to enable LFS and have admin enable hashed storage. More information
Assignee
Assign to
Algo 2020.06
Milestone
Algo 2020.06
Assign milestone
Time tracking
None
Due date
None
Reference: vidjil/vidjil#4334