feat: upgraded embedded metrics library for high resolution metrics

[jdoherty]

**Issue 1041 **

Description of changes:

Updated emf library allowing users to publish high resolution metrics

Checklist

• Meet tenets criteria
• Update tests
• Update docs
• PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

• Migration process documented
• Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Notes:

• No high resolution submitted for putSingleMetric
• Suggest we tackle the validation and removal of setAccessible (Remove usage of setAccessible #1449) in another ticket

[github-actions]

💾 Artifacts Size Report

Module	Version	Size (KB)
powertools-common	2.0.0-SNAPSHOT	9.59
powertools-serialization	2.0.0-SNAPSHOT	18.22
powertools-logging	2.0.0-SNAPSHOT	33.09
powertools-logging-log4j	2.0.0-SNAPSHOT	20.63
powertools-logging-logback	2.0.0-SNAPSHOT	17.04
powertools-tracing	2.0.0-SNAPSHOT	14.02
powertools-metrics	2.0.0-SNAPSHOT	14.07
powertools-parameters	2.0.0-SNAPSHOT	17.49
powertools-validation	2.0.0-SNAPSHOT	20.83
powertools-cloudformation	2.0.0-SNAPSHOT	17.01
powertools-idempotency-core	2.0.0-SNAPSHOT	35.57
powertools-idempotency-dynamodb	2.0.0-SNAPSHOT	12.92
powertools-large-messages	2.0.0-SNAPSHOT	17.48
powertools-batch	2.0.0-SNAPSHOT	16.89
powertools-parameters-ssm	2.0.0-SNAPSHOT	10.73
powertools-parameters-secrets	2.0.0-SNAPSHOT	9.93
powertools-parameters-dynamodb	2.0.0-SNAPSHOT	11.97
powertools-parameters-appconfig	2.0.0-SNAPSHOT	12.00

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.91%. Comparing base (82d4b30) to head (cda20c7).
Report is 66 commits behind head on v2.

❗ Current head cda20c7 differs from pull request most recent head 204dfe4

Please upload reports for the commit 204dfe4 to get more accurate results.

Additional details and impacted files

@@              Coverage Diff              @@
##                 v2    #1550       +/-   ##
=============================================
- Coverage     89.79%   76.91%   -12.89%     
- Complexity      406      466       +60     
=============================================
  Files            44       49        +5     
  Lines          1274     1685      +411     
  Branches        165      254       +89     
=============================================
+ Hits           1144     1296      +152     
- Misses           88      300      +212     
- Partials         42       89       +47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[sonarcloud]

Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[scottgerring]

Hey @jdoherty sorry for the high latency on this one! LGTM (and tests 🎸 )

Couple small things -->

I think the spotbugs warning needs an ignore in the overrides file:

[ERROR] Medium: Public static software.amazon.lambda.powertools.metrics.MetricsUtils.metricsLogger() may expose internal representation by returning MetricsUtils.metricsLogger [software.amazon.lambda.powertools.metrics.MetricsUtils] At MetricsUtils.java:[line 53] MS_EXPOSE_REP

TBH i'm not sure why this is being picked up now. You've not touched the class itself, but the implementation of metricsLogger has changed 🤷 Regardless, should be a quick fix.

Can you also chuck some words in the docs? The python guys have a section for high-resolution metrics that might be inspirational/reusable.

E2E tests are broken in v2 by my recent merge of the idempotency split. We can ignore that here.

[jeromevdl]

I think the spotbugs warning needs an ignore in the overrides file:

[ERROR] Medium: Public static software.amazon.lambda.powertools.metrics.MetricsUtils.metricsLogger() may expose internal representation by returning MetricsUtils.metricsLogger [software.amazon.lambda.powertools.metrics.MetricsUtils] At MetricsUtils.java:[line 53] MS_EXPOSE_REP

TBH i'm not sure why this is being picked up now. You've not touched the class itself, but the implementation of metricsLogger has changed 🤷 Regardless, should be a quick fix.

Fixed

Can you also chuck some words in the docs? The python guys have a section for high-resolution metrics that might be inspirational/reusable.

Done

@scottgerring, you can have a quick look but we should be good with that one.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[scottgerring]

LGTM! Merci @jdoherty and @jeromevdl