Skip to content

Conversation

rrg92
Copy link
Contributor

@rrg92 rrg92 commented Apr 16, 2019

Add a topic about impacts of CDC when patching instance. If instance enters on script upgrade mode, it will runs proc sp_vupgrade_replication, which runs sp_cdc_vupgrade. In this proc, it can rebuild indexes. I wokred on company that experinced this problem in SQL Server 2008 R2, and after talking with support, it was fixed, but current code (checked on version 2016), can trigger these rebuilds yet. BEcause this, i guess is important alert readers for they plan instance patching.

Add a topic about impacts of CDC when patching instance. If instance enters on script upgrade mode, it will runs proc sp_vupgrade_replication, which runs sp_cdc_vupgrade. In this proc, it can rebuild indexes. I wokred on company that experinced this problem in SQL Server 2008 R2, and after talking with support, it was fixed, but current code (checked on version 2016), can trigger these rebuilds yet. BEcause this, i guess is important alert readers for they plan instance patching.
@PRMerger13
Copy link
Contributor

@rrg92 : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@rothja
Copy link
Collaborator

rothja commented Apr 17, 2019

@rrg92 Thank you for the suggestion! Give me some time to review this and I'll get back to you soon.

@PRMerger7
Copy link
Contributor

@rothja : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@rothja
Copy link
Collaborator

rothja commented May 1, 2019

@rrg92 Sorry for the long delay. I did a little research and then went through your suggestion and made some formatting and wording changes to your change. I'm going to merge this in now. Thank you for your feedback and for taking time to improve the docs!

@rothja rothja merged commit 5ad53db into MicrosoftDocs:live May 1, 2019
@rrg92
Copy link
Contributor Author

rrg92 commented May 1, 2019

Nice @rothja! Thank you for attention to this.

@rrg92 rrg92 deleted the patch-2 branch May 2, 2019 03:29
@rrg92
Copy link
Contributor Author

rrg92 commented May 3, 2019

Hey @rothja, the merge was made and this change is on live branch, but not in the page. What is the frequency the page get synced with live branch?

@rothja
Copy link
Collaborator

rothja commented May 6, 2019

@rrg92 Thanks for the follow-up. We had a very rare publishing freeze last week. It is lifted today, and I expect you'll see the changes by tomorrow morning at the latest. Normally, public PRs go through in about 24 hours after they are merged in. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants