elastic / elastic/apm-agent-java

Remove duplicate metricset serialization logic

Open
#3,016 0 comments 0 reactions 0 assignees View on GitHub
agent-java technical-debt
Dominant language
Java
Stars
594
Forks
338
Avg merge
1d 13h
Merged PRs (30d)
25

Description

We currently have the logic for metricset serialization in three places:
* Internal metrics
* Micrometer metrics
* OpenTelemetry metrics

With #2968 we added the `MetricSetSerializer` class, which is used for the OpenTelemetry metricset serialization.
This class is generic enough that we should be able to extract a superclass which delegates just the `labels` serialization to its subclasses. We can then use this base class for all places where we serialize metricsets.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.