Skip to content

Commit

Permalink
reduce number of pr fetched in graphql query to avoid timeouts (#782)
Browse files Browse the repository at this point in the history
Summary:
[isl] reduce number of pr fetched in graphql query to avoid timeouts
Stack created with Sapling.
-> #782
#781
Pull Request resolved: #782

Test Plan: Imported from GitHub, without a Test Plan: line.

Reviewed By: quark-zju

Differential Revision: D51630746

Pulled By: evangrayk

fbshipit-source-id: a7c3189e02a4c6da772dd9001e6b20078b5d7747
  • Loading branch information
vuduchild authored and facebook-github-bot committed Nov 28, 2023
1 parent 72e707e commit ec38704
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion addons/isl-server/src/github/githubCodeReviewProvider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ export class GitHubCodeReviewProvider implements CodeReviewProvider {
// This is not very easy with github's graphql API, which doesn't allow more than 5 "OR"s in a search query.
// But if we used one-query-per-diff we would reach rate limiting too quickly.
searchQuery: `repo:${this.codeReviewSystem.owner}/${this.codeReviewSystem.repo} is:pr author:@me`,
numToFetch: 100,
numToFetch: 50,
});
if (allSummaries?.search.nodes == null) {
this.diffSummaries.emit('data', new Map());
Expand Down

0 comments on commit ec38704

Please sign in to comment.