wip: Optimization for cassandra extid reading to avoid an extra index
Closed
requested to merge generated-differential-D6263-source into generated-differential-D6263-target
wip because it does not work for now
Depends on !718 (closed)
Test Plan
tox (fails for now hence the wip stuff)
Migrated from D6263 (view on Phabricator)
Merge request reports
Activity
Filter activity
Build has FAILED
Patch application report for D6263 (id=22675)
Could not rebase; Attempt merge onto 589d20ed...
Merge made by the 'recursive' strategy. swh/storage/cassandra/cql.py | 60 ++++++++++++++++++++++++++++++++++++-- swh/storage/cassandra/storage.py | 36 ++++++++++++++++++----- swh/storage/in_memory.py | 40 ++++++++++++++++++++++--- swh/storage/interface.py | 20 +++++++++++-- swh/storage/postgresql/db.py | 33 +++++++++++++++++---- swh/storage/postgresql/storage.py | 32 +++++++++++++++++--- swh/storage/tests/storage_tests.py | 42 ++++++++++++++++++++++++++ 7 files changed, 238 insertions(+), 25 deletions(-)
Changes applied before test
commit 7b0ea794d2c06eda73fd47ee6784f592ecc0418c Merge: 589d20ed 5c7412fd Author: Jenkins user <jenkins@localhost> Date: Wed Sep 15 09:48:46 2021 +0000 Merge branch 'diff-target' into HEAD commit 5c7412fddf7391b25eb493e62007166bb27a0cf1 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Wed Sep 15 11:47:22 2021 +0200 wip: Optimization for cassandra extid reading to avoid an extra index wip because it does not work for now commit fc3d21489bf75058604873fa3629cd78da10c5e2 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Mon Sep 13 14:25:34 2021 +0200 Allow filtering extids per extid_version/extid_type when reading This impacts both the `extid_get_from_extid` and `extid_get_from_target` endpoints. Whe extid_version/extid_type are not provided, this keeps the existing behavior of returning all extids matching. Related to swh/infra/sysadm-environment#3567
Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1408/ See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1408/console
Discarded as the version that works got integrated into !718 (closed).
mentioned in merge request !718 (closed)
Please register or sign in to reply