[Cosmos] ArchBoard Review Items

[simorenoh]

Using this issue to keep track of items mentioned during the arch-board review. These are:

• GetAuthorizationHeader new method should be made private

• remove __aenter from any and all documentations

• PR for both of the above: [Cosmos] post-archboard fixes #24358

Others:

• open an issue to make refresh_routing_map_provider() and entire CosmosClientConnection object private: [Cosmos] Make CosmosClientConnection object private #24359
• conversation around the enable_cross_partition_query option and functionality in sync vs. async clients
• conversation on naming of read_conflict() vs get_conflict() - going with get, changes in the offer PR below
• look into post_trigger_include method option and see whether it needs to be deprecated in sync client/ included in async one

We're also still missing merging these PRs:

• [Cosmos] use python3 typehints and move options to kwargs on async client #23933
• [Cosmos] mark populate_query_metrics flag to be deleted when used and remove docs #22264
• [Cosmos] deprecate offer naming in methods - merge available #24428

[simorenoh]

PR for first of the two: #24358

[simorenoh]

• look into post_trigger_include method option and see whether it needs to be deprecated in sync client/ included in async one

For this one, this functionality is something we should support, and as such should remain in the async and sync clients. More on the rest api functionality on this if needed: https://docs.microsoft.com/rest/api/cosmos-db/triggers

[simorenoh]

• naming of read_conflict() vs get_conflict()

For this one, I also found that all READ calls in the scripts.py file are named as 'get_xxx' even though they make network calls and incur in RU usage - with this in mind, we should ensure we have the same standard all around.

[simorenoh]

4.3.0 Released https://pypi.org/project/azure-cosmos/4.3.0/