perf(graphqlmetrics): reduce allocated memory - Githubissues

wundergraph / cosmo

The open-source solution to building, maintaining, and collaborating on GraphQL Federation at Scale. The alternative to Apollo Studio and GraphOS.

https://cosmo-docs.wundergraph.com/

Apache License 2.0

742 stars 105 forks source link

perf(graphqlmetrics): reduce allocated memory #1356

Closed Noroth closed 1 week ago

Noroth commented 1 week ago

Motivation and Context

The memory consumption increased due to recent changes. This PR attempts to include some optimizations in that regard.

Additional enhancements

Prepare batch function now uses best effort approach to save as much data as possible in case of unexpected errors.
Errors will be logged instead of returned

Checklist

[ ] I have discussed my proposed changes in an issue and have received approval to proceed.
[ ] I have followed the coding standards of the project.
[ ] Tests or benchmarks have been added or updated.
[ ] Documentation has been updated on https://github.com/wundergraph/cosmo-docs.
[ ] I have read the Contributors Guide.