Skip to content

chore: Aot support for Logging and Metrics (WIP) #556

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from

Conversation

hjgraca
Copy link
Contributor

@hjgraca hjgraca commented Feb 21, 2024

Please provide the issue number

Issue number: #212

Summary

  • Logging AOT support
  • Metrics AOT support

Changes

  • Logging
    • Remove Logging lambda context reflection
    • Add PowertoolsSourceGenerationContext for types discovery at compile time
  • Metrics
    • Add MetricsSerializationContext for types discovery at compile time

Checklist

Please leave checklist items unchecked if they do not apply to your change.

Is this a breaking change?

RFC issue number:

Checklist:

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

Acknowledgment

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

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@auto-assign auto-assign bot requested a review from amirkaws February 21, 2024 14:23
@boring-cyborg boring-cyborg bot added area/common Core Powertools utility area/logging Core logging utility labels Feb 21, 2024
@auto-assign auto-assign bot requested a review from sliedig February 21, 2024 14:23
@boring-cyborg boring-cyborg bot added area/metrics Core metrics utility area/parameters area/tracing Core tracing utility tests labels Feb 21, 2024
@pull-request-size pull-request-size bot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Feb 21, 2024
@hjgraca hjgraca added do-not-merge PRs that are blocked for varying reasons aot labels Feb 21, 2024
@github-actions github-actions bot added the internal Maintenance changes label Feb 21, 2024
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
20.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@hjgraca hjgraca closed this Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aot area/common Core Powertools utility area/logging Core logging utility area/metrics Core metrics utility area/parameters area/tracing Core tracing utility do-not-merge PRs that are blocked for varying reasons internal Maintenance changes size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant