Skip to content

feat: expose more metrics for S3 - #1011

Merged
azkrishpy merged 12 commits into
mainfrom
expose-getters
Sep 28, 2026
Merged

azkrishpy merged 12 commits into
mainfrom
expose-getters

Conversation

@azkrishpy

@azkrishpy azkrishpy commented Aug 17, 2026 •

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:
Adds/fixes S3 request telemetry exposed via S3RequestMetrics:

  • getServiceId() — now returns "S3" (was "s3"), matching SDK convention
  • getServiceEndpoint() — now returns a full scheme-qualified URL (e.g. https://bucket.s3.region.amazonaws.com) instead of a bare host, so it can be parsed with URI.create()
  • getTimeToFirstByte() / getTimeToLastByte() — now return durations (delta from request start), not raw timestamps
  • getConnectionAcquisitionDurationNs() — new, time spent acquiring a connection for the request
  • getHttpManagerMetrics() — new, snapshot of the endpoint's connection pool state (available/pending/leased concurrency) taken just before this request asks for a connection
  • S3Client.getMaxActiveConnections() — new, the connection concurrency cap of the client.
  • Updates submodules to latest versions.

Requires awslabs/aws-c-s3#673 for newly bound client level API.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@azkrishpy
azkrishpy marked this pull request as ready for review September 10, 2026 18:52
@DmitriyMusatkin

Copy link
Copy Markdown
Contributor

description of changes? title of pr does not seem to match with whats in the diff?

@azkrishpy azkrishpy changed the title Expose headers for S3 Metrics feat: expose more metrics for S3 Sep 11, 2026
Comment thread src/main/java/software/amazon/awssdk/crt/s3/S3Client.java Outdated
@azkrishpy
azkrishpy merged commit 2d9d4d6 into main Sep 28, 2026
236 of 253 checks passed
@azkrishpy
azkrishpy deleted the expose-getters branch September 28, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants