Active-Active Region Application Failover #1034
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@AlonMagrafta I finished the style review. We just need the Python script in order to be ready to merge, right? |
Yossi and Yuval (Inbar) wrote this doc. We still need to upload a python script as stated in line 59.
bmansheim
force-pushed
the
AlonMagrafta-Region-Application-Failover
branch
from
699eb45 to
d9fa255
Compare
bmansheim
added
blocked
|
@banker I added the script to the branch but couldn't add it to the page in a way to download nicely. |
|
@AlonMagrafta is this something we need to complete before 6.0.20? |
|
@kaitlynmichael we can do this after 6.0.20, and once it's ready merge and publish, it's not release related |
AlonMagrafta
requested review from
kaitlynmichael
and removed request for
bmansheim
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
| Note that this document does not suggest that Redis pub/sub is reliable in the common sense. Messages can get lost in certain conditions, but that is acceptable in this case because typically the application determines that replication is down only after not being able to deliver a number of messages over a period of time. | ||
| {{< /note >}} | ||
|
|
||
| When you use the pub/sub data type to detect failures: |
There was a problem hiding this comment.
Where are the instructions for using the pub/sub this way?
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
content/rs/references/developing-for-active-active/region-application-failover-active-active.md
Outdated
Show resolved
Hide resolved
kaitlynmichael
approved these changes
Mar 26, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Yossi and Yuval (Inbar) wrote this doc.
We still need to upload a python script as stated in line 59.
@bmansheim let's coordinate before going live with that one. Thanks