Skip to content

Replaced code that returns the "instance_name"#3686

Merged
jborsecnik merged 1 commit intoMicrosoftDocs:livefrom
Rob-S:patch-3
Dec 11, 2019
Merged

Replaced code that returns the "instance_name"#3686
jborsecnik merged 1 commit intoMicrosoftDocs:livefrom
Rob-S:patch-3

Conversation

@Rob-S
Copy link
Copy Markdown
Contributor

@Rob-S Rob-S commented Nov 25, 2019

Even on a standalone instance, just like on an FCI instance mentioned in the "FCI_network_name" paragraph above it, the "SELECT @@ServerName" would return the "full replica name", not just the "instance_name" as the paragraph currently states. The type of instance makes no difference in what "SELECT @@ServerName" returns - always the "full replica name" - nor in the "InstanceName" property - always the instance name, or null for an unnamed "default server instance".

Even on a standalone instance, just like on an FCI instance mentioned in the "FCI_network_name" paragraph above it, the "SELECT @@ServerName" would return the "full replica name", not just the "instance_name" as the paragraph currently states. The type of instance makes no difference in what "SELECT @@ServerName" returns - always the "full replica name" - nor in the "InstanceName" property - always the instance name, or null for an unnamed "default server instance".
@PRMerger20
Copy link
Copy Markdown
Contributor

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

@MikeRayMSFT
Copy link
Copy Markdown
Contributor

@Rob-S - thank you for the submission. Reviewing.
@kfarlee - Please review.

@jborsecnik
Copy link
Copy Markdown
Contributor

@MikeRayMSFT, okay to merge?

@MikeRayMSFT
Copy link
Copy Markdown
Contributor

#sign-off

@MikeRayMSFT
Copy link
Copy Markdown
Contributor

@jborsecnik - it's ready to merge. #sign-off

@jborsecnik jborsecnik merged commit 5a09360 into MicrosoftDocs:live Dec 11, 2019
@Rob-S Rob-S deleted the patch-3 branch December 17, 2019 16:25
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.

6 participants