Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

feat/implementation of parallel clients limit #103

Merged
merged 11 commits into from
Oct 30, 2021

Conversation

amanenk
Copy link
Contributor

@amanenk amanenk commented Oct 14, 2021

fixes #56

Added parameter that limits resources parallel fetch

@amanenk amanenk marked this pull request as ready for review October 14, 2021 15:43
@amanenk amanenk requested a review from roneli October 19, 2021 12:33
@roneli roneli changed the title implementation of paralel clients limit feat/implementation of parallel clients limit Oct 19, 2021
Copy link
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would add tests to this use case.

Copy link
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice test

@roneli roneli merged commit d48afc8 into cloudquery:main Oct 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support parallelization rate limit
2 participants