From 6698b2f06435fd8da738bd3e03cc949913f9c1ff Mon Sep 17 00:00:00 2001 From: Mindaugas Vinkelis Date: Wed, 11 Dec 2024 08:10:58 +0200 Subject: [PATCH 1/2] Update CHANGELOGs for time changes in metrics aggregations --- opentelemetry-sdk/CHANGELOG.md | 4 ++++ opentelemetry-stdout/CHANGELOG.md | 2 ++ 2 files changed, 6 insertions(+) diff --git a/opentelemetry-sdk/CHANGELOG.md b/opentelemetry-sdk/CHANGELOG.md index 95714301aa..a348cc5cb3 100644 --- a/opentelemetry-sdk/CHANGELOG.md +++ b/opentelemetry-sdk/CHANGELOG.md @@ -2,6 +2,10 @@ ## vNext +- *Breaking* `start_time` and `time`, in moved from DataPoints to aggregations (Sum, Gauge, Histogram, ExpoHistogram) see [#2377](https://github.com/open-telemetry/opentelemetry-rust/pull/2377) and [#2411](https://github.com/open-telemetry/opentelemetry-rust/pull/2411). + +- *Breaking* `start_time` is no longer optional for `Sum` aggregation, see [#2367](https://github.com/open-telemetry/opentelemetry-rust/pull/2367), but is still optional for `Gauge` aggregation see [#2389](https://github.com/open-telemetry/opentelemetry-rust/pull/2389). + - *Breaking* - SimpleLogProcessor modified to be generic over `LogExporter` to avoid dynamic dispatch to invoke exporter. If you were using diff --git a/opentelemetry-stdout/CHANGELOG.md b/opentelemetry-stdout/CHANGELOG.md index 536381e24e..f6eb70dcde 100644 --- a/opentelemetry-stdout/CHANGELOG.md +++ b/opentelemetry-stdout/CHANGELOG.md @@ -2,6 +2,8 @@ ## vNext +- *Breaking* time fields, `StartTime` and `EndTime` is printed on aggregation (Sum, Gauge, Histogram, ExpoHistogram) with 2 tabs, previously it was on aggregation data point, with 3 tabs, see [#2377](https://github.com/open-telemetry/opentelemetry-rust/pull/2377) and [#2411](https://github.com/open-telemetry/opentelemetry-rust/pull/2411). + ## 0.27.0 Released 2024-Nov-11 From 60b074d4f25190eec01fd47f2bce16ac210e7c38 Mon Sep 17 00:00:00 2001 From: Mindaugas Vinkelis Date: Thu, 12 Dec 2024 15:22:20 +0200 Subject: [PATCH 2/2] Update opentelemetry-sdk/CHANGELOG.md point out that this is for exporter authors only Co-authored-by: Cijo Thomas --- opentelemetry-sdk/CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opentelemetry-sdk/CHANGELOG.md b/opentelemetry-sdk/CHANGELOG.md index a348cc5cb3..07c80decda 100644 --- a/opentelemetry-sdk/CHANGELOG.md +++ b/opentelemetry-sdk/CHANGELOG.md @@ -2,7 +2,7 @@ ## vNext -- *Breaking* `start_time` and `time`, in moved from DataPoints to aggregations (Sum, Gauge, Histogram, ExpoHistogram) see [#2377](https://github.com/open-telemetry/opentelemetry-rust/pull/2377) and [#2411](https://github.com/open-telemetry/opentelemetry-rust/pull/2411). +- *Breaking(Affects custom metric exporter authors only)* `start_time` and `time` is moved from DataPoints to aggregations (Sum, Gauge, Histogram, ExpoHistogram) see [#2377](https://github.com/open-telemetry/opentelemetry-rust/pull/2377) and [#2411](https://github.com/open-telemetry/opentelemetry-rust/pull/2411), to reduce memory. - *Breaking* `start_time` is no longer optional for `Sum` aggregation, see [#2367](https://github.com/open-telemetry/opentelemetry-rust/pull/2367), but is still optional for `Gauge` aggregation see [#2389](https://github.com/open-telemetry/opentelemetry-rust/pull/2389).