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

Harvester improvements. #68

Merged
merged 1 commit into from
Jul 12, 2021
Merged

Harvester improvements. #68

merged 1 commit into from
Jul 12, 2021

Conversation

a-feld
Copy link
Contributor

@a-feld a-feld commented Jul 7, 2021

This PR does the following:

  • Adds a harvester tag to the user agent to measure use of the harvester struct
  • Uses a sync.WaitGroup to manage concurrent requests for each data type. This means that data will arrive quickly and context timeouts will be less likely.
  • Makes buildSplitRequests package internal

@alanwest alanwest merged commit 60c8d2b into master Jul 12, 2021
@alanwest alanwest deleted the harvester-improvements branch July 12, 2021 17:44
@alanwest alanwest restored the harvester-improvements branch July 12, 2021 17:44
@alanwest alanwest deleted the harvester-improvements branch July 12, 2021 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alanwest alanwest alanwest approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@a-feld @alanwest