Repository navigation
Narrower package/user queries - #1532
Merged
Merged
Conversation
Call sites that only need part of the package index now ask the core feature for exactly that, rather than fetching the whole index: - single-package lookups use queryLookupPackageName - new queryAllPackageNames, queryAllPackageIds and queryCountPackages serve callers that need names, IDs or the package count - new queryLatestUploadsBetween serves the notification job's recent uploads and revisions The acid store implements the new Store fields over the in-memory index, so behaviour is unchanged.
Authentication no longer needs the whole user database: it takes a UserLookup with name and API token lookups, and the privilege checks drop the user database argument they never used. The users store gains lookups by id, name and token, listings of all and active users, and a user name search. Callers that looked up a single user, listed users, or searched by name use these instead of fetching the whole user database.
Contributor
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
This was referenced Oct 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To continue the work in #1524 by @tomjaguarpaw, this converts queries that take the whole
database into queries that could conceivably be implemented using efficient SQL
queries, once more database backends are added.