Open jeet1995 opened 2 months ago
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @kushagraThapar @pjohari-ms @TheovanKraay.
@jeet1995 @kushagraThapar Found below issues in CosmosDiagnostics.
One suggestion for the showing the options in the diagnostics is only showing the effective options, so which one were actually executed in the request regardless of client, dynamic, or request level.
Tasks
[ ] From incidents and customer concerns, the stringified representation of
CosmosDiagnostics
occupies space which breaches size / character limits in whatever log analytics service the customer may choose to use, causing diagnostic strings to be truncated - an example of such limits specific to Azure Monitor - https://learn.microsoft.com/en-us/azure/azure-monitor/service-limits#logs-ingestion-api. Evaluate ways to reduce the size / characters taken up by a stringified representation ofCosmosDiagnostics
instance without losing the information it wishes to convey.[ ] As a part of the above issue, evaluate if there is a way to rejig the diagnostic string to display errors first as opposed to successes.
[ ] The diagnostic string captures configurations set at the client-level or through operation-specific
Cosmos*RequestOptions
instances. With, dynamic request options - see #40061, operation-specificCosmos*RequestOptions
instances can be overridden. It could help to understand the source of these settings in the diagnostics.[ ] Expose
CosmosDiagnostics
forhandleChanges
andhandleLatestVersionChanges
overloads exposed forChangeFeedProcessor
. ForhandleLatestVersionChanges
, bothALL_VERSIONS_AND_DELETES
andINCREMENTAL
change feed modes should emitCosmosDiagnostics
. Validate the correctness of the diagnostics emitted by thequeryChangeFeed()
API.