Skip to content

Commit e384201

Browse files
committed
feat(bulk-import): optimize findAllRepositories to fetch imported and all repositories concurrently
Signed-off-by: Dominik Augustín <daugusti@redhat.com>
1 parent 539070e commit e384201

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

  • workspaces/bulk-import/plugins/bulk-import-backend/src/service/handlers/repository

workspaces/bulk-import/plugins/bulk-import-backend/src/service/handlers/repository/repositories.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,17 @@ export async function findAllRepositories(
5858
}',${pageNumber},${pageSize})..`,
5959
);
6060

61-
const alreadyImportedRepositories =
62-
await deps.catalogHttpClient.listCatalogUrlLocations();
61+
const [alreadyImportedRepositories, allRepositoriesResponse] =
62+
await Promise.all([
63+
deps.catalogHttpClient.listCatalogUrlLocations(),
64+
deps.gitApiService.getRepositoriesFromIntegrations(search, pageSize),
65+
]);
66+
6367
const alreadyImportedRepositoriesLocationTargets = Array.from(
6468
new Set(alreadyImportedRepositories.uniqueCatalogUrlLocations.keys()),
6569
);
6670

67-
const { repositories: allRepositories, errors } =
68-
await deps.gitApiService.getRepositoriesFromIntegrations(search, pageSize);
71+
const { repositories: allRepositories, errors } = allRepositoriesResponse;
6972

7073
const notImportedYetRepositories = allRepositories.filter(repo => {
7174
const html_urlWithSlash = repo.html_url.concat('/');

0 commit comments

Comments
 (0)