-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merge existing changes to the forked version #1
Conversation
34e7d89
to
bc3192f
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1 +/- ##
=========================================
Coverage ? 92.22%
=========================================
Files ? 116
Lines ? 30252
Branches ? 0
=========================================
Hits ? 27899
Misses ? 2353
Partials ? 0 ☔ View full report in Codecov by Sentry. |
5073285
to
902935f
Compare
902935f
to
9d93769
Compare
* [GROW-2938] do not reset redis_connection on an error * [GROW-2938] add backoff to more errors * [GROW-2938] recover from SlotNotCoveredError * [GROW-2938] prevent get_node_from_slot from failing due to concurrent cluster slots refresh * [GROW-2938] add retry to ClusterPipeline (cherry picked from commit 63e06dd)
* [GROW-2938] do not reset redis_connection on an error * [GROW-2938] add backoff to more errors * [GROW-2938] recover from SlotNotCoveredError * [GROW-2938] prevent get_node_from_slot from failing due to concurrent cluster slots refresh * [GROW-2938] add retry to ClusterPipeline (cherry picked from commit 63e06dd)
* [GROW-2938] do not reset redis_connection on an error * [GROW-2938] add backoff to more errors * [GROW-2938] recover from SlotNotCoveredError * [GROW-2938] prevent get_node_from_slot from failing due to concurrent cluster slots refresh * [GROW-2938] add retry to ClusterPipeline (cherry picked from commit 63e06dd)
* [GROW-2938] do not reset redis_connection on an error * [GROW-2938] add backoff to more errors * [GROW-2938] recover from SlotNotCoveredError * [GROW-2938] prevent get_node_from_slot from failing due to concurrent cluster slots refresh * [GROW-2938] add retry to ClusterPipeline (cherry picked from commit 63e06dd)
Pull Request check-list
Please make sure to review and check all of these items:
$ tox
pass with this change (including linting)?NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
I'm migrating changes i've made in Soda to the forked redis-py. This should remove most of the duplicated codes in the Soda.
I'll be merging the changes to the master first and backport the changes to 4.4 and 4.5 branch later to release the custom version
This is a public repository, so i've not migrated the Soda specific codes