Skip to content

Commit

Permalink
Add alternative
Browse files Browse the repository at this point in the history
Signed-off-by: Arve Knudsen <[email protected]>
  • Loading branch information
aknuds1 committed Jul 1, 2024
1 parent 7c8f527 commit 5cfeef3
Showing 1 changed file with 13 additions and 2 deletions.
15 changes: 13 additions & 2 deletions proposals/2024-04-10-native-support-for-info-metrics-metadata.md
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,20 @@ TODO:

## Alternatives

The section stating potential alternatives. Highlight the objections reader should have towards your proposal as they read it. Tell them why you still think you should take this path [[ref](https://twitter.com/whereistanya/status/1353853753439490049)]
### Add metadata as prefixed labels

1. This is why not solution Z...
Instead of encoding metadata, e.g. OTel resource attributes, as info metric labels, add them directly as labels to corresponding metrics.

#### Pros

* Simplicity, removes need for joining with info metrics

#### Cons

* Metrics will have potentially far more labels than what's strictly necessary to identify them
* Temporary series churn when migrating existing metrics to this new scheme
* Increased series churn when metadata labels change
* More labels per metric increases CPU/memory usage

## Action Plan

Expand Down

0 comments on commit 5cfeef3

Please sign in to comment.