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.
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
optimization for x-repo search index builds #1466
optimization for x-repo search index builds #1466
Conversation
98b99da
to
8f82ed4
Compare
8f82ed4
to
4d3eab0
Compare
… version. fixes pulp#1467 Signed-off-by: James Tanner <tanner.jc@gmail.com>
4d3eab0
to
fc590ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments/questions, but nothing really blocking.
if previous_version is None: | ||
return None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean with retain_repo_versions = 1
there is no optimization?
fixes pulp#1467 Signed-off-by: James Tanner <tanner.jc@gmail.com>
0bb05e2
to
31d0a78
Compare
Backport to 0.17: 💚 backport PR created✅ Backport PR branch: Backported as #1469 🤖 @patchback |
https://issues.redhat.com/browse/AAH-2337
https://issues.redhat.com/browse/AAH-2351
#1467
In the previous iteration of this code, ALL collection indexes would be rebuilt if at least one of their CVs were still in the repository version. This patch uses the .added and .removed functions to make a set of namespace(s).name(s) which have changed in the newest repository version. This set of namespace(s).names(s) can then be used to skip rebuilding the index for a collection which hasn't changed in some way.
Without the patch:
With the patch: