Page MenuHomeSoftware Heritage

sql/archiver: add the initialization script for the archiver database
AbandonedPublic

Authored by qcampos on Jul 8 2016, 3:48 PM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers
Summary

Keep the script used to initialize the tables used for the archival.

Diff Detail

Repository
rDSTO Storage manager
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 276
Build 405: Software Heritage Python tests
Build 404: arc lint + arc unit

Event Timeline

qcampos retitled this revision from to sql/archiver: add the initialization script for the archiver database.
qcampos updated this object.
qcampos edited the test plan for this revision. (Show Details)
qcampos added a reviewer: ardumont.
ardumont edited edge metadata.
ardumont added a subscriber: olasd.
ardumont added inline comments.
sql/archiver/content_archive_initialize.sql
3

Like i noted on T412, we need to change the identifier.
Sorry i did not see it earlier.

This text of 5 (for Banco) multiplied by the number of contents (one per row in content_archive), too much lost space.
A simple integer should be enough.

9

@olasd pointed me to the right file (concerning the backup-ed content in banco) so this is no longer true.

Well, at least, when T412 is done, we'll be able to inject only what's really missing.
Also, we'd be better off adding the status as enum (also to reduce storage size).

This revision now requires changes to proceed.Jul 9 2016, 10:17 AM
sql/archiver/content_archive_initialize.sql
9

status as enum (also to reduce storage size).

which is already the case ^^

@qcampos Can you please close it? Thanks in advance. It's no longer relevant (work is done on T412 for now :)