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

Update sys-sql-modules-transact-sql.md - Unnecessary object_name invocation #6688

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
DanielAdeniji:patch-15
Aug 23, 2021
Merged

Update sys-sql-modules-transact-sql.md - Unnecessary object_name invocation #6688
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
DanielAdeniji:patch-15

Conversation

@DanielAdeniji
Copy link
Copy Markdown
Contributor

Peers:-

It is preferable to directly use the object name from sys.objects rather than to invoke object_name function.

The reasons include:-

  1. Metadata locks during object creation
  2. The existence of sys.objects in the query, as is

Also, added sys.schemas so that we can include the schema name, as well.

Peers:-

It is preferable to directly use the object name from sys.objects rather than to invoke object_name function.

The reasons include:-

1) Metadata locks during object creation
2) The existence of sys.objects in the query, as is

Also, added sys.schemas so that we can include the schema name, as well.
@PRMerger17
Copy link
Copy Markdown
Contributor

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

@PRMerger9
Copy link
Copy Markdown
Contributor

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

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

Thanks @DanielAdeniji for submitting this PR! I have added a commit to support your updates to the sample query and refresh other style things in the article. Your commit will be merged and this article update is in flight. After the updated article is published, GitHub will list you as an article contributor.

Thanks again for this feedback, your suggestions help improve our documentation.

#sign-off

@ktoliver ktoliver merged commit f84dc35 into MicrosoftDocs:live Aug 23, 2021
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.

7 participants