Improved wording and removed redundant sentences #22
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request. |
|
Thank you @biancaghiurutan |
fenxu
pushed a commit
that referenced
this pull request
Dec 6, 2018
* Update server-performance-and-activity-monitoring.md * Update monitor-and-tune-for-performance.md * Replace non-ASCII characters with their ASCII counterparts. Ellipsis, N-dash, M-dash. (metadata 1387353) * Update rotate-always-encrypted-keys-using-powershell.md * AzureRM version to get StorageContainerSASToken * Fixing typo "ervice" vs "service" * fixed typo * Update monitor-and-tune-for-performance.md * Update monitor-and-tune-for-performance.md * Update toc.yml Fixing tree missing item * Update monitor-and-tune-for-performance.md * Update use-dmvs-determine-usage-performance-views.md * Update monitor-sql-server-components.md * Update update-statistics-transact-sql.md * Update update-statistics.md * Update update-statistics-task-maintenance-plan.md * Update sp-updatestats-transact-sql.md * removed locale from download url removed '/en-us' locale from download url * Update update-statistics-transact-sql.md * Remove promise of row sequence in STRING_SPLIT, fixing PR 8229. * RevoScaleR formatting, links, consolidation * Update toc.yml moving CE to internals * Update cursors.md * Acrolinx improvements. * Update cursors.md * Update cursors.md * Update cursors.md * Update toc.yml * Update cursors.md * Update declare-cursor-transact-sql.md * Update close-transact-sql.md * Update deallocate-transact-sql.md * Update fetch-transact-sql.md * Update cursors.md * Update declare-cursor-transact-sql.md * fix broken links from public repo * Test * edit file, new ss * Public contribution, broken link test. * test * Update query-profiling-infrastructure.md * Update create-a-new-plan-guide.md * Update create-a-new-plan-guide.md * fix link * metadata ms.technology, DevO 1387353, ado/ - d-q-s/ * edits * edits * removed locale per reviewer feedback * Edits on articles 9-14 * Reverted numeric prefix added for navigation * Update quickstart-sql-database.md * Update quickstart-sql-database.md * Clarified the support statement for MI * Spelling * Metadata, DevO 1387353, docs/database-engine/ (follows PR 8289) * Cleaned up one sentence recently added by public PR. * Cleaned up one sentence added recently by customer PR. * Update alter-database-scoped-configuration-transact-sql.md * DB setup edits * Manually fix non-ASCII converts to ???, checkmark (to Yes), Russian (retained). * Nellie edits to Java perms for Linux and windows * Better retention of Russian German etc, instead of convert to ???. * Formatting * removed unused prereqs * promoted Jar creation to H2 and moved down * Updating the query store topics to include SQL DW * TOC edit * Scorecard updates to a topic * ms rs 1129 updates remove 2016, add pag rpts to rb * ms rs 1129 updates fix include errors * ms rs 1129 updates fix include again * ms rs 1129 updates fix include try 3 * Replace absolute link * Fix link * Update the-transaction-log-sql-server.md * update back slashes to forward slashes * Fixing broken links to retired BLOGS, DevO 1341794, rel-dab/in-memory-oltp/ * Added link to index.md * Removed line from blog- that wrongly claimed another link to another blog. * Changed example C to match result * TRANSLATE vs REPLACE I realised, after making a now accepted change, that actually `TRANSLATE` and multiple `REPLACE` statements are not equivalent. If a character is replaced to one that is also appears later in the translatations character string, the character will not be replaced. For `REPLACE`, however, this is not true. `REPLACE` will replace all instances of that character, including those that were replaced in one of the nested functions. I have therefore added this information in the Remarks section. * Fixed file name, fixed some links and titles, minor edits for brevity * edits * Acro validation * Update quickstart-sql-database.md Minor edits * Update quickstart-sql-database.md minor edit * edits * Update big-data-cluster-overview.md Fixed casing inconsistency * Update big-data-cluster-overview.md Sorry, I thought "big data cluster" was the product name. I fixed it. * edits * Sumit TOC feedback * Swap MLS with main landing page for MLS feature area * titles * Fix typo, remove version of SSDT Fix typo Remove version of SSDT - seems pretty pointless having version no as it gets changed so often nowadays * Not available in SQL 2014 , only 2016 * Metadata, DevO 1387353, docs/dea/ -thru- docs/integration-services/ * Diacritic 'a' needed. And in ssis-catalog.md an utterly invisible unknown non-ASCII char. * Metadata, DevO 1387353, docs/linux/ -thru- docs/master-data-services/ * Bad non-ASCII invisible chars, even VS Code extension complained. * ?? fixes if Non-ASCII * Typo * Metadata, DevO 1399287 (continues 1387353), docs/mdx/ thru docs/relational-databases/. 2. * Metadata, DevO 1399287 (1387353), docs/samples/ thru docs/xquery/ EXCEPT NOT docs/t-sql/. * Metadata, DevO 1399287 (1387353), docs/t-sql/. * 'ms.workload:' metadata Removals (manual), DevO 1387353. * ssas1203 * edits alter login * Update quickstart-sql-database.md Minor edits * added more context to example * Update quickstart-sql-database.md header typo * [sql server] mi support * Update quickstart-sql-database.md Fix header err * Update quickstart-sql-database.md Fix header typo I hope * Update quickstart-sql-database.md header err * Update quickstart-sql-database.md Changed 'click' to select * Update quickstart-sql-database.md Minor edits * Update quickstart-sql-database.md minor edit * adjust markdown syntax * Update sql-server-management-studio-changelog-ssms.md Added link for QTA docs page * Update sql-server-management-studio-changelog-ssms.md * Managed Instance cannot change collation * Minor acorlynx fixes * Update sql-server-management-studio-changelog-ssms.md * M32847: One of the "https://" should be "http://" Hello, @maggiesMSFT, Translator has reported possible source content issue. Please review and merge the suggested proposed file change to avoid this error. If you make related fix in another PR, then share your PR number, so we can confirm and close this PR. Many thanks in advance. * Add steps on how to produce output in DMV SQL 2014 and SQL 2016 steps * Removed links to MSDN blog being retired, batch 1. * Update sys-dm-os-wait-stats-transact-sql.md * Removed links to MSDN blog being retired, batch 2. * Doc bug fixes for big data clusters * Removed links to MSDN blog being retired, batch 3.
fenxu
pushed a commit
that referenced
this pull request
Nov 21, 2020
…_objects 20201117 1217 polybase character lengths may be larger than expected
fenxu
pushed a commit
that referenced
this pull request
May 5, 2022
Update create-availability-group-transact-sql.md
fenxu
pushed a commit
that referenced
this pull request
Dec 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.