Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Geearl/5872 fr internal error #140

Merged
merged 9 commits into from
Jul 24, 2023
Merged

Conversation

georearl
Copy link
Contributor

@georearl georearl commented Jul 21, 2023

This PR covers tickets 5872, 5874, 5877, 5873 and 5880. The functionality introduced includes:

  • working around internal errors we encounter in FR when it is at capacity
  • fixed skipped file types registering as errors
  • moved all backoff and retry variables to configuration settings
  • changed logging so it only writes out to cosmos after the completion of the function with all status updates, not on every status update. This will reduce the amount of write calls by about 10:1, and therefore reduce the chances of throttling and cost
  • documentation of known issues around throttling and internal errors plus how to override the backoff and retry settings for development purposes

@dayland dayland changed the base branch from main to vNext-Dev July 23, 2023 22:56
@georearl georearl merged commit 31564fe into vNext-Dev Jul 24, 2023
2 checks passed
@georearl georearl deleted the geearl/5872-FR-internal-error branch July 24, 2023 14:54
lmwilki pushed a commit that referenced this pull request Aug 7, 2023
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