Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhance query performance using batch-loading #881

Merged
merged 6 commits into from
Aug 21, 2023
Merged

Enhance query performance using batch-loading #881

merged 6 commits into from
Aug 21, 2023

Commits on Aug 18, 2023

  1. Use batch-loading for organisation fields

    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    0ac1559 View commit details
    Browse the repository at this point in the history

  2. Avoid hidden database-select by peewee 

    Accessing `box.location.base_id` previously issued another DB select
query to fetch the location field albeit already joining with the
location table. Now the `Box.select()` call is updated
    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    b7deeda View commit details
    Browse the repository at this point in the history

  3. Make more use of base-loader

    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    27b1f79 View commit details
    Browse the repository at this point in the history

  4. Make more use of user-loader

    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    93ada55 View commit details
    Browse the repository at this point in the history

  5. Introduce SimpleDataLoader class 

    - reduce duplicated code
- sort classes in file
    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    5388be7 View commit details
    Browse the repository at this point in the history

  6. Use work-around to use SimpleDataLoader for batch-loading product cat… 

    …egories

Would be fixed by updating the RBP names in the Auth0 scripts for dev
and staging; but that in turn after merging confusingly breaks all open
PRs as long as they are not rebased on master yet
Cf. https://trello.com/c/J34Z923v
    @pylipp
    pylipp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    d3e10c1 View commit details
    Browse the repository at this point in the history