-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[MAINTENANCE] Add retries to requests
in usage stats integration tests
#4600
[MAINTENANCE] Add retries to requests
in usage stats integration tests
#4600
Conversation
✅ Deploy Preview for niobium-lead-7998 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
def requests_session_with_retries() -> requests.Session: | ||
# https://stackoverflow.com/a/35636367 | ||
session = requests.Session() | ||
retries = Retry(total=5, backoff_factor=1, status_forcelist=[502, 503, 504]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like how the backoff_factor
works! We should keep an eye on how often we are using this though for performance reasons.
…thub.com/great-expectations/great_expectations into bugfix/GREAT-725/fix-regex-param-builder * 'bugfix/GREAT-725/fix-regex-param-builder' of https://github.com/great-expectations/great_expectations: [FEATURE] Implement Multi-Column Domain Builder for Rule-Based Profiler (#4604) [MAINTENANCE] Add retries to `requests` in usage stats integration tests (#4600)
…sts (#4600) * feat: add retry fixture * chore: add type hinting
* develop: docs: fix snippet ref (#4609) [BUGFIX] Adjust output of `regex` ParameterBuilder to match Expectation (#4594) [FEATURE] Rule-Based Profiler: ColumnPairDomainBuilder (#4608) chore : update example notebook for RBP to include multicolumn domain builder (#4606) Pass random seed to bootstrap estimator (#4605) [MAINTENANCE] Miscellaneous test cleanup (#4602) [FEATURE] Implement Multi-Column Domain Builder for Rule-Based Profiler (#4604) [MAINTENANCE] Add retries to `requests` in usage stats integration tests (#4600) [MAINTENANCE] Standardize DomainBuilder Constructor Arguments Ordering (#4599) [MAINTENANCE] Instrument package dependencies (#4583) [BUGFIX] Adjust output of datetime `ParameterBuilder` to match Expectation (#4590)
Please annotate your PR title to describe what the PR does, then give a brief bulleted description of your PR below. PR titles should begin with [BUGFIX], [FEATURE], [DOCS], or [MAINTENANCE]. If a new feature introduces breaking changes for the Great Expectations API or configuration files, please also add [BREAKING]. You can read about the tags in our contributor checklist.
Changes proposed in this pull request:
Definition of Done
Please delete options that are not relevant.
Thank you for submitting!