Skip to content

Commit

Permalink
rename Record class in metrics sdk to Accumulation
Browse files Browse the repository at this point in the history
  • Loading branch information
Azfaar Qureshi committed Nov 10, 2020
1 parent ad01a56 commit 13df04c
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions opentelemetry-sdk/src/opentelemetry/sdk/metrics/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ class ValueObserver(Observer, metrics_api.ValueObserver):
"""See `opentelemetry.metrics.ValueObserver`."""


class Record:
class Accumulation:
"""Container class used for processing in the `Processor`"""

def __init__(
Expand Down Expand Up @@ -382,10 +382,10 @@ def _collect_metrics(self) -> None:
bound_instrument,
) in metric.bound_instruments.items():
for view_data in bound_instrument.view_datas:
record = Record(
accumulation = Accumulation(
metric, view_data.labels, view_data.aggregator
)
self.processor.process(record)
self.processor.process(accumulation)

if bound_instrument.ref_count() == 0:
to_remove.append(labels)
Expand All @@ -404,8 +404,8 @@ def _collect_observers(self) -> None:
continue

for labels, aggregator in observer.aggregators.items():
record = Record(observer, labels, aggregator)
self.processor.process(record)
accumulation = Accumulation(observer, labels, aggregator)
self.processor.process(accumulation)

def record_batch(
self,
Expand Down

0 comments on commit 13df04c

Please sign in to comment.