Added note from SET XACT_ABORT#7494
Conversation
Extra note from SET XACT_ABORT page seems important as it is the best reason to prefer THROW over RAISERROR, and so far not mentioned
|
@jonathanmbaker : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #label:"aq-pr-triaged" |
|
Hi @jonathanmbaker , Thanks for creating this PR. Could you please review the suggestion made by @Court72 ? The links you provided are great, but for localization reasons we don't commit the location signifier (in this case "en-us") in URLs. If you are not familiar with the GitHub suggestion feature, it should allow you to commit the suggestion from the 'files changed' tab on this PR. Thanks again for submitting this PR, |
|
@Court72 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Thanks, @LitKnd. I went ahead and committed the update. If the changes are ready for publication, add a Thanks! |
|
Aha, thanks @Court72 , I misunderstood your previous comment. That looks great. @jonathanmbaker , please ignore my prior comment -- you don't need to accept the suggestion. Thanks again for submitting this change. Kendra from the Database Docs Team #sign-off |
Extra note from SET XACT_ABORT page seems important as it is the best reason to prefer THROW over RAISERROR, and so far not mentioned
"The THROW statement honors SET XACT_ABORT. RAISERROR does not. New applications should use THROW instead of RAISERROR."