-
Notifications
You must be signed in to change notification settings - Fork 480
[docs] SQL Server HA failover #33747
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
base: main
Are you sure you want to change the base?
Conversation
3a9ad39
to
f6e013f
Compare
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 vote we add a disclaimer about async vs. sync availability mode to let users know about the potential for inconsistency with async.
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.
Could we link to the HA section from the note in the ingest data section (that way we let people know that it exists and that the
https://github.com/MaterializeInc/materialize/pull/33747/files#diff-e5e71ad1b8a944d3f24689fefa0d148dd43bc4842ebbf56d59e958281d1061baR241
Thank you both. Addressed. |
If I'm reading this section correctly, manual intervention may be needed in the case that there are only 2 replicas in an AG. (or if all but one database fails). This may be worth calling out as well. |
@tylerhartwig that was part of a section as containing best practices. I want to avoid copying to much of microsofts docs over. |
Motivation
Tips for reviewer
Checklist
$T ⇔ Proto$T
mapping (possibly in a backwards-incompatible way), then it is tagged with aT-proto
label.