Skip to content

[PR #4471/207655ff backport][3.36] Sync optimization: do existing content check in first stage#4473

Merged
dralley merged 1 commit into
3.36from
patchback/backports/3.36/207655ff537671b6ffc2d01e55c03527d4768b98/pr-4471
Jun 3, 2026
Merged

[PR #4471/207655ff backport][3.36] Sync optimization: do existing content check in first stage#4473
dralley merged 1 commit into
3.36from
patchback/backports/3.36/207655ff537671b6ffc2d01e55c03527d4768b98/pr-4471

Conversation

@patchback
Copy link
Copy Markdown

@patchback patchback Bot commented Jun 3, 2026

This is a backport of PR #4471 as merged into main (207655f).

Build a cache of existing packages from the previous repository version. In the event that the repository being created is substantially similar (very likely), this should reduce the amount of database queries required in the QueryExistingContent stage, which can discriminate between Content objects yet-to-be-saved and ones which were already saved.

Assisted-By: claude-opus-4.6

📜 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

Build a cache of existing packages from the previous repository version.
In the event that the repository being created is substantially similar
(very likely), this should reduce the amount of database queries
required in the QueryExistingContent stage, which can discriminate
between Content objects yet-to-be-saved and ones which were already
saved.

Assisted-By: claude-opus-4.6
(cherry picked from commit 207655f)
@dralley dralley merged commit 8dab4da into 3.36 Jun 3, 2026
23 of 25 checks passed
@dralley dralley deleted the patchback/backports/3.36/207655ff537671b6ffc2d01e55c03527d4768b98/pr-4471 branch June 3, 2026 14:54
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.

1 participant