Avoid costly metrics collection when metric collector is NoOpMetricCo…

[croudet]

…llector

Description

Avoid costly metrics collection when metric collector is NoOpMetricCollector.

Motivation and Context

Profiling with visualvm shows that collecting metrics takes resources.

Checklist

• [ x] I have read the CONTRIBUTING document
• [ x] Local run of mvn install succeeds
• [ x] My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• [ x] I have read the README document
• I have added tests to cover my changes
• [ x] All new and existing tests passed
• A short description of the change has been added to the CHANGELOG
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• [ x] I confirm that this pull request can be released under the Apache 2 license

[zoewangg]

Thank you for your PR! LGTM

Could you add a changelog entry? You can do so by running the scripts/new-change script and following the instructions and then committing the new file created by the script in .changes/next-release with your changes.

(Make sure the commits are squashed into one)

[croudet]

Done.

[zoewangg]

Thank you for your contribution!

[codecov-io]

Codecov Report

Merging #2174 (6c3ffe4) into master (3a103f3) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #2174   +/-   ##
=========================================
  Coverage     77.44%   77.44%           
  Complexity      328      328           
=========================================
  Files          1222     1222           
  Lines         38304    38304           
  Branches       3018     3018           
=========================================
+ Hits          29663    29665    +2     
+ Misses         7185     7184    -1     
+ Partials       1456     1455    -1     

Flag	Coverage Δ	Complexity Δ
unittests	77.44% <0.00%> (+<0.01%)	0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ	Complexity Δ
.../amazon/awssdk/core/internal/util/MetricUtils.java	86.95% <0.00%> (ø)	0.00 <0.00> (ø)
...ty/internal/IdleConnectionCountingChannelPool.java	87.80% <0.00%> (+2.43%)	0.00% <0.00%> (ø%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a103f3...6c3ffe4. Read the comment docs.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

85.7% Coverage
0.0% Duplication