Skip to content

Mention limits of concurrency#3307

Merged
ktoliver merged 17 commits into
MicrosoftDocs:masterfrom
AlexGuteniev:master
Aug 18, 2021
Merged

Mention limits of concurrency#3307
ktoliver merged 17 commits into
MicrosoftDocs:masterfrom
AlexGuteniev:master

Conversation

@AlexGuteniev

Copy link
Copy Markdown
Contributor

Mention limits of concurrency. See microsoft/STL#2099

@PRMerger14

Copy link
Copy Markdown
Contributor

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

@PRMerger19

Copy link
Copy Markdown
Contributor

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

@PRMerger10

Copy link
Copy Markdown
Contributor

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

@PRMerger14

Copy link
Copy Markdown
Contributor

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

@PRMerger6

Copy link
Copy Markdown
Contributor

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

@PRMerger6

Copy link
Copy Markdown
Contributor

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

@ktoliver ktoliver added the aq-pr-triaged Tracking label for the PR review team label Aug 16, 2021

@TylerMSFT TylerMSFT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @AlexGuteniev. I appreciate the Microsoft-specific updates.
Could you smooth the sentences out a little bit. They sound fragmentary. For example, instead of "Maximums number of chunks..." it would help if you wrote "The maximum number of chunks..." This pattern should be repeated everywhere that you have a fragmentary sentence, e.g. "Number of concurrent threads is..." could be "The number of concurrent threads is...", etc.

@PRMerger20

Copy link
Copy Markdown
Contributor

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

@PRMerger20 PRMerger20 requested a review from TylerMSFT August 16, 2021 18:36
@PRMerger15

Copy link
Copy Markdown
Contributor

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

@PRMerger17

Copy link
Copy Markdown
Contributor

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

@AlexGuteniev

Copy link
Copy Markdown
Contributor Author

@TylerMSFT, I tried, but may still don't get it right. My native language doesn't have the feature.

@TylerMSFT TylerMSFT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's good. Thank you. One final thing: could you update the ms.date of the changed files to reflect that they've been updated. Then I'll sign off. Really appreciate you doing this work.

@PRMerger17 PRMerger17 requested a review from TylerMSFT August 17, 2021 05:13
@PRMerger17

Copy link
Copy Markdown
Contributor

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

@PRMerger8

Copy link
Copy Markdown
Contributor

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

@PRMerger14

Copy link
Copy Markdown
Contributor

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

@TylerMSFT TylerMSFT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AlexGuteniev - thank you very much for your excellent contribution to the Microsoft standard library docs!

@TylerMSFT

Copy link
Copy Markdown
Collaborator

#sign-off

@PRMerger10 PRMerger10 requested a review from TylerMSFT August 18, 2021 15:28
@PRMerger10

Copy link
Copy Markdown
Contributor

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

@PRMerger6

Copy link
Copy Markdown
Contributor

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

@PRMerger20

Copy link
Copy Markdown
Contributor

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

@ktoliver ktoliver merged commit 05be7ed into MicrosoftDocs:master Aug 18, 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.