Page MenuHomeSoftware Heritage

Have comments on all columns of all databases
Closed, MigratedEdits Locked

Description

Check the databases have the necessary comments on their column.

Services/modules concerned are:

  • swh-storage
  • swh-scheduler
  • swh-indexer

Each of those modules should have a folder sql containing an existing schema file.
This should be amended to add the necessary information (missing column description).

This implies also a sql/upgrade/<dbversion>.sql file with the diff amending the schema.
(One can use D1133 as a model to see how to proceed)

Event Timeline

Can you provide a few more details so I can work on this? Maybe which packages will be affected and what is expected in the comments.

modules swh-scheduler, swh-indexer, swh-storage, all seem to have column comments written in 30-swh-schema.sql

is there anything left to be done to close the task?

ardumont updated the task description. (Show Details)
ardumont added a project: Easy hack.

there seems to be an inconsistency between sql/upgrades and latest sql version in swh-storage. The latest upgrade is 136.sql while the version in 30-swh-schema.sql is 133. Should I name the next upgrade 137?

The latest upgrade is 136.sql while the version in 30-swh-schema.sql is 133. Should I name the next upgrade 137?

Nice catch.
That should indeed be in sync (for some epsilon of current diffs not landed).
So yeah, my take is that it should say current 136 in schema.sql.

All columns are already commented in swh-indexer

All columns commented in swh-scheduler, waiting review.
Some columns for swh-storage required a small discussion to frame appropriate comments.

D1582 has been pushed the task can be closed