Skip to content
This repository was archived by the owner on Aug 8, 2025. It is now read-only.

Started using timeit for measuring elapsed time#185

Closed
VitalyAssurance wants to merge 1 commit intolitl:masterfrom
VitalyAssurance:use_timeit2
Closed

Started using timeit for measuring elapsed time#185
VitalyAssurance wants to merge 1 commit intolitl:masterfrom
VitalyAssurance:use_timeit2

Conversation

@VitalyAssurance
Copy link
Copy Markdown

@VitalyAssurance VitalyAssurance commented Dec 19, 2022

There is some evidence based on Datadog traces that backoff library does not calculate elapsed time correctly. For example, there are occurrences where extra retries are made even though the elapsed time exceeds specified max_time parameter for backoff.on_exception decorator. I suspect the problem is due to system clock being adjusted for time drift, or a similar issue.

I propose to use timeit instead of datetime.datetime.now() for measuring elapsed time.

@VitalyAssurance VitalyAssurance closed this by deleting the head repository Jan 31, 2023
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.

1 participant