Add JMX metrics for S3 http client usage in native FS

[mayurjpatel]

When s3 client was changed to use AWS SDK v2 from AWS SDK v1, the set of s3 pool metrics were missed.

With this PR, these http pool metrics are now exposed via JMX beans. The reference for the http metrics exposed by AWS SDK can be found here:
https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/http/HttpMetric.html

Description

When s3 client was changed to use AWS SDK v2 from AWS SDK v1, the set of s3 pool metrics were missed.

With this PR, these http pool metrics are now exposed via JMX beans. The reference for the http metrics exposed by AWS SDK can be found here:
https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/http/HttpMetric.html

Additional context and related issues

#23944

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

[cla-bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Mayur Patel.
This is most likely caused by a git client misconfiguration; please make sure to:

1. check if your git client is configured with an email to sign commits git config --list | grep email
2. If not, set it up using git config --global user.email [email protected]
3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

[cla-bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

[zhaner08]

LGTM

[pettyjamesm]

I'm not sure these make sense as a DistributionStat since the connection pool stats of availableConcurrency, leasedConcurrency, and pendingConcurrency are point in time snapshot values and not values that are accumulated. In AwsSdkClientCoreStats these are recorded as AtomicLong values instead.

[mosabua]

Did you close this PR accidentally @mayurjpatel ?

[mayurjpatel]

Did you close this PR accidentally @mayurjpatel ?

Yeah, sorry, my mistake. I am going to open a new PR with @pettyjamesm 's feedback.

[mosabua]

Sounds good .. you probably can just reopen this one (or I can for you) .. as long as you are using the same branch and pushing to it (force pushing..)

[mayurjpatel]

Ended up creating #24843 after i sync'ed the fork. Apologies for the noise.