Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Added variable verbiage#5359

Merged
rothja merged 1 commit intoMicrosoftDocs:livefrom
way0utwest:patch-32
Aug 24, 2020
Merged

Added variable verbiage#5359
rothja merged 1 commit intoMicrosoftDocs:livefrom
way0utwest:patch-32

Conversation

@way0utwest
Copy link
Copy Markdown
Contributor

This statement does not effect local or table variables.

This statement does not effect local or table variables.
@PRMerger8
Copy link
Copy Markdown
Contributor

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

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Aug 24, 2020

@way0utwest Thanks for the suggested improvement. Do you have some sample T-SQL that shows the point you're making here? It looks like maybe something you've tested recently? That would help me confirm and approve. Thanks! -Jason

@way0utwest
Copy link
Copy Markdown
Contributor Author

This should help

DECLARE @founders VARCHAR(50) = 'way0utwest';
DROP TABLE IF EXISTS MyTable;
CREATE TABLE MyTable
(founder VARCHAR(50));
INSERT dbo.MyTable (founder) VALUES ('way0utwest');
BEGIN TRAN;
SET @founders = 'rothja';
UPDATE dbo.MyTable SET founder = 'rothja';
ROLLBACK;
SELECT @founders AS f;
SELECT * FROM dbo.MyTable AS mt;

The change inside the transaction and rollback do not apply as they would for live data.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Aug 24, 2020

@way0utwest Thanks! I'll verify on my side with your sample and approve the PR. Appreciate your time.

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Aug 24, 2020
@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Aug 24, 2020

#sign-off

@rothja rothja merged commit bf887b4 into MicrosoftDocs:live Aug 24, 2020
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