Skip to content
Snippets Groups Projects

wip: Optimization for cassandra extid reading to avoid an extra index

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 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).

  • Merge request was abandoned

  • mentioned in merge request !718 (closed)

Please register or sign in to reply
Loading