Page MenuHomeSoftware Heritage

Fix unique index definition for path to avoid error with large values
ClosedPublic

Authored by aeviso on Jan 31 2022, 1:01 PM.

Diff Detail

Event Timeline

Build is green

Patch application report for D7050 (id=25564)

Could not rebase; Attempt merge onto dfac07f890...

Updating dfac07f..f2993ce
Fast-forward
 requirements-swh.txt                 |  1 +
 sql/upgrades/003.sql                 | 15 ++++++++++++
 swh/provenance/__init__.py           |  9 ++++++-
 swh/provenance/archive.py            |  3 +--
 swh/provenance/postgresql/archive.py |  2 +-
 swh/provenance/sql/30-schema.sql     |  6 ++---
 swh/provenance/sql/60-indexes.sql    |  2 ++
 swh/provenance/storage/archive.py    |  2 +-
 swh/provenance/swhgraph/__init__.py  |  0
 swh/provenance/swhgraph/archive.py   | 46 ++++++++++++++++++++++++++++++++++++
 10 files changed, 78 insertions(+), 8 deletions(-)
 create mode 100644 sql/upgrades/003.sql
 create mode 100644 swh/provenance/swhgraph/__init__.py
 create mode 100644 swh/provenance/swhgraph/archive.py
Changes applied before test
commit f2993ce4785158350c141397d1f04e5531823fbd
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Mon Jan 31 12:56:30 2022 +0100

    Fix unique index definition for path to avoid error with large values

commit f2301c188729ca1d67618cb05edb27f976a4914d
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Jan 20 16:08:32 2022 +0100

    Add partial implementation of `ArchiveGraph` class

commit f2cbe1c38fc652c0335a882d2f0982bc3943aaf4
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Jan 20 15:35:43 2022 +0100

    Remove ordered result constrain from `snapshot_get_heads`
    
    It is not require anymore after simplifying the origin-revision
    layer algorithm.

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/566/ for more details.

Build is green

Patch application report for D7050 (id=25566)

Could not rebase; Attempt merge onto dfac07f890...

Updating dfac07f..c348456
Fast-forward
 MANIFEST.in                              |  4 ++-
 requirements-swh.txt                     |  1 +
 sql/upgrades/003.sql                     | 15 +++++++++++
 swh/provenance/__init__.py               |  9 ++++++-
 swh/provenance/archive.py                |  3 +--
 swh/provenance/postgresql/archive.py     |  2 +-
 swh/provenance/sql/10-superuser-init.sql |  3 +++
 swh/provenance/sql/30-schema.sql         |  6 ++---
 swh/provenance/sql/60-indexes.sql        |  1 +
 swh/provenance/storage/archive.py        |  2 +-
 swh/provenance/swhgraph/__init__.py      |  0
 swh/provenance/swhgraph/archive.py       | 46 ++++++++++++++++++++++++++++++++
 swh/provenance/tests/conftest.py         |  7 +++--
 swh/provenance/tests/test_cli.py         |  3 +++
 14 files changed, 91 insertions(+), 11 deletions(-)
 create mode 100644 sql/upgrades/003.sql
 create mode 100644 swh/provenance/sql/10-superuser-init.sql
 create mode 100644 swh/provenance/swhgraph/__init__.py
 create mode 100644 swh/provenance/swhgraph/archive.py
Changes applied before test
commit c348456f8ab501399d8bb775f55b6b576ca3a154
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Mon Jan 31 12:56:30 2022 +0100

    Fix unique index definition for path to avoid error with large values

commit f2301c188729ca1d67618cb05edb27f976a4914d
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Jan 20 16:08:32 2022 +0100

    Add partial implementation of `ArchiveGraph` class

commit f2cbe1c38fc652c0335a882d2f0982bc3943aaf4
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Thu Jan 20 15:35:43 2022 +0100

    Remove ordered result constrain from `snapshot_get_heads`
    
    It is not require anymore after simplifying the origin-revision
    layer algorithm.

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/567/ for more details.

Build is green

Patch application report for D7050 (id=25567)

Rebasing onto dfac07f890...

Current branch diff-target is up to date.
Changes applied before test
commit bbab4c65c1b6b91cb994e49a198721c604d09c4e
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Mon Jan 31 12:56:30 2022 +0100

    Fix unique index definition for path to avoid error with large values

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/568/ for more details.

ardumont added a subscriber: ardumont.

lgtm, couple of remarks/suggestions inline.

sql/upgrades/003.sql
7

that sql command won't work if not used as admin.
I think it's ok to drop it from the migration script.
(keep it in the next 10-superuser-init.sql though).

swh/provenance/tests/conftest.py
109 ↗(On Diff #25567)

drop the print ;)

swh/provenance/tests/test_cli.py
102 ↗(On Diff #25567)

can't you reuse the fixture defined earlier?

This revision is now accepted and ready to land.Jan 31 2022, 4:52 PM
aeviso added inline comments.
swh/provenance/tests/test_cli.py
102 ↗(On Diff #25567)

that fixture has an already initialized database, it's not suitable for this particular test

This revision was landed with ongoing or failed builds.Jan 31 2022, 4:59 PM
This revision was automatically updated to reflect the committed changes.
aeviso marked an inline comment as done.

Build is green

Patch application report for D7050 (id=25573)

Rebasing onto dfac07f890...

Current branch diff-target is up to date.
Changes applied before test
commit 4dd24fd5aba526b116cd98c593fd2ba7c2866fd2
Author: Andres Ezequiel Viso <aeviso@softwareheritage.org>
Date:   Mon Jan 31 12:56:30 2022 +0100

    Fix unique index definition for path to avoid error with large values

See https://jenkins.softwareheritage.org/job/DPROV/job/tests-on-diff/572/ for more details.