[PULP-1668] Optimize re-sync with content caching#7754
Conversation
|
Looks good! A few things to look at, maybe we can make this even more efficient |
|
Measured benefit using https://fixtures.pulpproject.org/file-perf/ (20k files) with immediate policy sync: Without patch: With patch: The biggest impact comes from the Syncing with on-demand policy is already fast and is not significantly affected by this change (Sync time: 92 seconds, Re-sync time: 42 seconds). |
|
@dralley I will squash the commits before merging, but I am keeping them separate for easier review now. |
a05fba5 to
c4c68af
Compare
|
@jobselko Keep the The rest can be squashed and stay in this PR. Great work! This is a huge performance improvement 🚀 👍 |
c4c68af to
c54ccc1
Compare
|
@jobselko Can you squash commits 1, 2, and 4, and address the merge conflict? I think it's just the |
|
Discussed with @dralley that we are not going to merge this today. |
9f40b0d to
37ce3a4
Compare
0c357d0 to
ce6948f
Compare
ce6948f to
b91cb48
Compare
Assisted By: Claude Opus 4.6
b91cb48 to
637cdc1
Compare
Related to #7558
📜 Checklist
See: Pull Request Walkthrough