Fix and Update Telemetry Properties and Tests #456
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.
Description:
This PR significantly refactors telemetry data management in the SDK and enhances tests to ensure more efficient and reliable tracking of HTTP request metrics.
Key Changes:
Telemetry Refactor:
ThreadLocal<Map<String, Object>>
withThreadLocal<RequestMetrics>
for a more structured telemetry data management.RequestMetrics
class with dedicated methods for generating and formatting telemetry headers.ApacheHttpClientTransport
with a centralisedhandleException
method.Telemetry API Enhancements:
prev_request_id
: ID of the previous request.request_id
: Unique ID of the current request.prev_request_duration
: Duration of the previous request in milliseconds.toTelemetryHeader
method inRequestMetrics
.Extended Tests:
Fixes:
Impact:
Additional Notes:
ApacheHttpClientTransport
.@Disabled
are available for manual testing under specific scenarios.