-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Bug Fix - SQL Server HADR queries for SQL Versions #8833
Bug Fix - SQL Server HADR queries for SQL Versions #8833
Conversation
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.
🤝 ✅ CLA has been signed. Thank you!
Sorry this is beyond my SQL capabilities... :-) |
Thank you for the fix! Looks good to me! |
…mments for description
closes #8750 |
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.
Just minor questions but other than that looks good
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.
The code looks good as it is now, I've added the explanation.
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.
Ok thanks for the clarification, happy to merge it now then
Thank you @Trovalo for helping with the clarifications and approval. And thank you @helenosheaa for merging the PR. I'll resolve the open conversation in the above thread. |
Required for all PRs:
This PR fixes a bug in SqlServer plugin's HADR queries where various SQL Server versions (from 2012 onwards) were not handled, and causes the plugin to report failures for 2012 and 2014 versions.
closes #8750