AddSuportForLoggingEmptyPageDiagnostics- shortTermFix #26869
Merged
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.
Issue:
For a simple cross partition query, the request diagnostics for empty pages will not included in the cosmos diagnostics.
This PR is a short term fix which gives customer a way to enable the request diagnostics logging for empty pages, which will help in high latency scenarios investigation.
Fix:
Add support for empty page request diagnostics logging through system property.
Example:
System.setProperty("COSMOS.QUERY_EMPTY_PAGE_DIAGNOSTICS_ENABLED","true");
2022-02-04 09:38:01,000 [cosmos-rntbd-nio-2-4] INFO com.azure.cosmos.implementation.query.ParallelDocumentQueryExecutionContext - Empty page request diagnostics for correlatedActivityId [324c9cd0-85e1-11ec-bd6a-79ed8d05be53] - activityId [328e87d6-85e1-11ec-bd6a-79ed8d05be53] - [[{"userAgent":"azsdk-java-cosmos/4.26.0-beta.2 MacOSX/10.16