Cleaned up Azure::Core::Context API surface

[LarryOsterman]

Updated semantics of Azure::Core::Context

1. ApplicationContext is deprecated.
2. A new constructor for Context is added to enable directly creating a context with a deadline.

Updated documentation for Azure::Core::Context to express the current semantics.

Also generalized documentation cleanup.

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

• [ ] C++ Guidelines
• [ ] Doxygen docs
• [ ] Unit tests
• [ ] No unwanted commits/changes
• [ ] Descriptive title/description
  • [ ] PR is single purpose
  • [ ] Related issue listed
• [ ] Comments in source
• [ ] No typos
• [ ] Update changelog
• [ ] Not work-in-progress
• [ ] External references or docs updated
• [ ] Self review of PR done
• [ ] Any breaking changes?

[LarryOsterman]

/azp run cpp - storage - ci

[azure-pipelines[bot]]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[LarryOsterman]

/azp run cpp - keyvault - ci

[azure-pipelines[bot]]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[LarryOsterman]

/azp run cpp - tables - ci

[azure-pipelines[bot]]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[LarryOsterman]

/azp run cpp - identity - ci

[azure-pipelines[bot]]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[LarryOsterman]

Ping. At this point, I'm happy with the design and would appreciate any feedback.

Thanks!

[LarryOsterman]

Ping :).

[LarryOsterman]

Ping :).

[LarryOsterman]

Ping again. I would like to get this in soon, so would appreciate feedback.

[LarryOsterman]

Ping again.

[ahsonkhan]

I'll make time to review and page-back the conversation tomorrow, so I can provide more meaningful feedback.

From a PR review perspective, I trust that you've got the specifics well. The higher order question, that would be great for us to get consensus on, is about the design and end user experience. A PR conversation is likely not the best medium for that discussion (will discuss offline).

[LarryOsterman]

I'll make time to review and page-back the conversation tomorrow, so I can provide more meaningful feedback.

From a PR review perspective, I trust that you've got the specifics well. The higher order question, that would be great for us to get consensus on, is about the design and end user experience. A PR conversation is likely not the best medium for that discussion (will discuss offline).

Agreed that PR comments aren't the right place for this, but to help frame the discussion: What parts of this change do you feel require additional discussion?

[LarryOsterman]

Ping.

[LarryOsterman]

/azp run cpp - core - ci

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).

[LarryOsterman]

Ping. I believe that as of the discussion on 6/27, this should be the implementation of the final design.

[LarryOsterman]

/azp run cpp - storage - ci

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).

[LarryOsterman]

Ping. Waiting on feedback from @EmmaZhu, @Jinming-Hu , @RickWinter , @ahmadmsft, @gkostal , @anilba06 , @antkmsft , @vinjiang , and @microzchang.

This PR cannot proceed until every one of you signs off on it.