Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 864b644

Browse files
committed
fix observer
1 parent adce43e commit 864b644

File tree

1 file changed

+7
-3
lines changed
  • opentelemetry-sdk/src/opentelemetry/sdk/metrics

1 file changed

+7
-3
lines changed

opentelemetry-sdk/src/opentelemetry/sdk/metrics/__init__.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,11 @@
2525
from opentelemetry.sdk.metrics.export.aggregate import Aggregator
2626
from opentelemetry.sdk.metrics.export.batcher import Batcher
2727
from opentelemetry.sdk.metrics.export.controller import PushController
28-
from opentelemetry.sdk.metrics.view import ViewData, ViewManager
28+
from opentelemetry.sdk.metrics.view import (
29+
get_default_aggregator,
30+
ViewData,
31+
ViewManager,
32+
)
2933
from opentelemetry.sdk.resources import Resource
3034
from opentelemetry.sdk.util import get_dict_as_key
3135
from opentelemetry.sdk.util.instrumentation import InstrumentationInfo
@@ -255,13 +259,13 @@ def __init__(
255259
def observe(
256260
self, value: metrics_api.ValueT, labels: Dict[str, str]
257261
) -> None:
258-
key = get_labels_as_key(labels)
262+
key = get_dict_as_key(labels)
259263
if not self._validate_observe(value, key):
260264
return
261265

262266
if key not in self.aggregators:
263267
# TODO: how to cleanup aggregators?
264-
self.aggregators[key] = ObserverAggregator()
268+
self.aggregators[key] = get_default_aggregator(self)
265269
aggregator = self.aggregators[key]
266270
aggregator.update(value)
267271

0 commit comments

Comments
 (0)