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

Avoid fetching thousands of pages from GitLab API #377

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

neverpanic
Copy link

When you comment on an issue in a project that has thousands of issues, or operate on a merge request in a project with thousands of MRs, did attempts to fetch them all to identify the ID of the one you interacted with.

This is OK for projects that are small in size, but becomes a huge issue for large projects such as https://gitlab.com/gitlab-org/gitlab, which has 218k issues.

Check for the number of pages that would be fetched and avoid loading them all if there are more than 20 pages. This will lead to issue and merge request IDs missing from the report, but it will allow the report to finish within your lifetime.

When you comment on an issue in a project that has thousands of issues,
or operate on a merge request in a project with thousands of MRs, did
attempts to fetch them all to identify the ID of the one you interacted
with.

This is OK for projects that are small in size, but becomes a huge issue
for large projects such as https://gitlab.com/gitlab-org/gitlab, which
has 218k issues.

Check for the number of pages that would be fetched and avoid loading
them all if there are more than 20 pages. This will lead to issue and
merge request IDs missing from the report, but it will allow the report
to finish within your lifetime.

Signed-off-by: Clemens Lang <cllang@redhat.com>
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.

1 participant