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

Update sql-server-linux-configure-mssql-conf.md#550

Merged
GitHubber17 merged 2 commits intoMicrosoftDocs:livefrom
pradm:patch-1
Apr 18, 2018
Merged

Update sql-server-linux-configure-mssql-conf.md#550
GitHubber17 merged 2 commits intoMicrosoftDocs:livefrom
pradm:patch-1

Conversation

@pradm
Copy link
Copy Markdown
Contributor

@pradm pradm commented Apr 18, 2018

Updating the SQL Agent section to make sure users are aware , if mssql.conf file does not have an entry for enabled under [sqlagent] section we internally assume sql agent needs to be enabled and start the PE process.

@PRMerger16
Copy link
Copy Markdown
Contributor

@pradm : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Apr 18, 2018

@pradm Thanks for the contribution. I see you closed this. If you closed it to make a different change, note that you can edit the change in the browser from the "Files Changed" tab in the same PR.

Is this Pradeep from Microsoft? Thanks!

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Apr 18, 2018

Reopening per @pradm.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Apr 18, 2018

@pradm Thanks again. I have a small change to suggest before I sign off. Can you change your sentence to the following:

If **sqlagent.enabled** is not present in the mssql.conf settings file, then SQL Server internally assumes that SQL Server Agent is enabled.

@PRMerger12
Copy link
Copy Markdown
Contributor

@pradm : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Apr 18, 2018

@pradm Looks great. #sign-off

@GitHubber17 GitHubber17 merged commit de36d01 into MicrosoftDocs:live Apr 18, 2018
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.