Skip to content

API calls retrieving all results may be slow#195

Merged
timols merged 1 commit into
timols:masterfrom
johnou:optimize_retrieve_all
Mar 12, 2017
Merged

API calls retrieving all results may be slow#195
timols merged 1 commit into
timols:masterfrom
johnou:optimize_retrieve_all

Conversation

@johnou

@johnou johnou commented Mar 7, 2017

Copy link
Copy Markdown
Contributor

Motivation:

Speed up API calls.

Modifications:

Set maximum per page param when retrieving all results.

Result:

Less calls to GitLab API endpoint when retrieving all results.

Fixes #162.

@johnou

johnou commented Mar 7, 2017

Copy link
Copy Markdown
Contributor Author

@timols PTAL

Motivation:

Speed up API calls.

Modifications:

Set maximum per page param when retrieving all results.

Result:

Less calls to GitLab API endpoint when retrieving all results.
@johnou
johnou force-pushed the optimize_retrieve_all branch from 7874651 to a271d4c Compare March 9, 2017 18:29
@timols
timols merged commit 27eb69b into timols:master Mar 12, 2017
@johnou
johnou deleted the optimize_retrieve_all branch March 12, 2017 19:24
@johnou

johnou commented Mar 12, 2017

Copy link
Copy Markdown
Contributor Author

@timols thanks! When do you plan on dropping 1.2.8?

@johnou

johnou commented Mar 16, 2017

Copy link
Copy Markdown
Contributor Author

@timols poke

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.

2 participants