Skip to content

Improve content_ids handling by defered loading - #7867

Open
mdellweg wants to merge 1 commit into
pulp:mainfrom
mdellweg:content_ids_sql
Open

Improve content_ids handling by defered loading#7867
mdellweg wants to merge 1 commit into
pulp:mainfrom
mdellweg:content_ids_sql

Conversation

@mdellweg

Copy link
Copy Markdown
Member

The array_field can be huge and most of the time we don't need the values in python. Some uses can even be handled by clever SQL without ever channelling all the data through the network.

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@mdellweg
mdellweg marked this pull request as ready for review July 24, 2026 15:15
@github-actions github-actions Bot removed the wip label Jul 24, 2026
@mdellweg mdellweg changed the title WIP Improve content_ids handling by defered loading Improve content_ids handling by defered loading Jul 27, 2026
@mdellweg
mdellweg requested a review from gerrod3 July 27, 2026 15:20

@pedro-psb pedro-psb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was having a look into this, I think we could avoid hydrating content_ids for other methods as well, like added, removed and contains.

The array_field can be huge and most of the time we don't need the
values in python. Some uses can even be handled by clever SQL without
ever channelling all the data through the network.
@acheng-01

Copy link
Copy Markdown

We have some overlapping changes here... Would you prefer that you merge this in first and then I rebase on the changes or should I undo my changes related to the deferment and have you rebase instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants