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.

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

When writing main solv file (primary.xml) don't store filelists #1752

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

kontura
Copy link
Contributor

@kontura kontura commented Oct 3, 2024

In a repo filelists are stored in different repodata, limit the written repodata when writing main solv file.

Otherwise the filelists are stored twice: in the main solv file and in the filelists specific solvx file. (This happens when downloading both the primary.xml and filelists.xml at the same time.)

It also improves performance because when filelists were part of main solv file they were always loaded (regardless of the optional_metadata_types option).

Closes: #1749

In a repo filelists are stored in different repodata, limit the written
repodata when writing main solv file.

Otherwise the filelists are stored twice: in the main solv file and in the
filelists specific solvx file. (This happens when downloading both the
primary.xml and filelists.xml at the same time.)

It also improves performance because when filelists were part of main
solv file they were always loaded (regardless of the
`optional_metadata_types` option).
@m-blaha m-blaha self-assigned this Oct 3, 2024
Copy link
Member

@m-blaha m-blaha left a comment

Choose a reason for hiding this comment

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

Thank you! The change looks reasonable and according to my testing it works fine. Unfortunately the parts of libsolv this relies on is basically undocumented, so fingers crossed there won't be any unexpected negative side effects.

@m-blaha m-blaha added this pull request to the merge queue Oct 3, 2024
Merged via the queue into rpm-software-management:main with commit b2a5461 Oct 3, 2024
17 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filelists are stored twice in solv file cache
2 participants