Skip to content

Update dml-triggers.md#7215

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
AntonC9018:patch-2
Jan 25, 2022
Merged

Update dml-triggers.md#7215
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
AntonC9018:patch-2

Conversation

@AntonC9018
Copy link
Copy Markdown
Contributor

On line 49, I'm confused as to whether the trigger actually gets to do the final insert itself, or if it just allows or denies it. I feel like "before inserting the record into the payroll table" implies that, at the end of the day, trigger is the one doing the inserting. It should be changed to something like "before the insertion of the record happens/takes place/finishes/succeeds/goes through" if it doesn't do the insert itself. Am I wrong?

On line 49, I'm confused as to whether the trigger actually gets to do the final insert itself, or if it just allows or denies it. `before inserting the record into the payroll table`, I feel like, implies that at the end of the trigger is the one doing the inserting. It should be changed to something like `before the insertion of the record happens/takes place/finishes/succeeds/goes through` if it doesn't do the insert itself. Am I wrong?
@PRMerger20
Copy link
Copy Markdown
Contributor

@AntonC9018 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Copy Markdown
Contributor

#label:"aq-pr-triaged"

@PRMerger20 PRMerger20 added the aq-pr-triaged tracking label for the PR review team label Jan 19, 2022
@MikeRayMSFT
Copy link
Copy Markdown
Contributor

@AntonC9018 - thank you for the submission. It corrects some language problems with the article as is.

In the case of an INSTEAD OF trigger, the command of the trigger is executed in lieu of the command that launched the trigger, so the current example:

a trigger can be defined to either produce an error message and roll back the transaction, or insert a new record into an audit trail before inserting the record into the payroll table

The trigger completes the defined command.

If that doesn't answer the question, please ask at SQL Server Q&A.

#sign-off

@AntonC9018
Copy link
Copy Markdown
Contributor Author

@MikeRayMSFT makes sense, thank you!

@ktoliver ktoliver merged commit e97f721 into MicrosoftDocs:live Jan 25, 2022
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